Skip to content

Commit b5be333

Browse files
committed
- Fixed the build (sorry)
- Removed unused method SVN: trunk@5069
1 parent e4e6b2b commit b5be333

File tree

2 files changed

+55
-68
lines changed

2 files changed

+55
-68
lines changed

src/NHibernate/Linq/Functions/FunctionRegistry.cs

Lines changed: 54 additions & 67 deletions
Original file line numberDiff line numberDiff line change
@@ -12,67 +12,54 @@ public class FunctionRegistry : ILinqToHqlGeneratorsRegistry
1212
{
1313
public static readonly ILinqToHqlGeneratorsRegistry Instance = new FunctionRegistry();
1414

15-
private readonly Dictionary<MethodInfo, IHqlGeneratorForMethod> registeredMethods = new Dictionary<MethodInfo, IHqlGeneratorForMethod>();
16-
private readonly Dictionary<MemberInfo, IHqlGeneratorForProperty> registeredProperties = new Dictionary<MemberInfo, IHqlGeneratorForProperty>();
17-
private readonly List<IHqlGeneratorForType> typeGenerators = new List<IHqlGeneratorForType>();
15+
private readonly Dictionary<MethodInfo, IHqlGeneratorForMethod> registeredMethods = new Dictionary<MethodInfo, IHqlGeneratorForMethod>();
16+
private readonly Dictionary<MemberInfo, IHqlGeneratorForProperty> registeredProperties = new Dictionary<MemberInfo, IHqlGeneratorForProperty>();
17+
private readonly List<IHqlGeneratorForType> typeGenerators = new List<IHqlGeneratorForType>();
1818

19-
private FunctionRegistry()
20-
{
21-
// TODO - could use reflection here
22-
Register(new QueryableGenerator());
23-
Register(new StringGenerator());
24-
Register(new DateTimeGenerator());
25-
Register(new ICollectionGenerator());
26-
}
27-
28-
private bool GetMethodGeneratorForType(MethodInfo method, out IHqlGeneratorForMethod methodGenerator)
29-
{
30-
methodGenerator = null;
31-
32-
foreach (var typeGenerator in typeGenerators.Where(typeGenerator => typeGenerator.SupportsMethod(method)))
33-
{
34-
methodGenerator = typeGenerator.GetMethodGenerator(method);
35-
return true;
36-
}
37-
return false;
38-
}
39-
40-
private bool GetStandardLinqExtensionMethodGenerator(MethodInfo method, out IHqlGeneratorForMethod methodGenerator)
41-
{
42-
methodGenerator = null;
19+
private FunctionRegistry()
20+
{
21+
// TODO - could use reflection here
22+
Register(new QueryableGenerator());
23+
Register(new StringGenerator());
24+
Register(new DateTimeGenerator());
25+
Register(new ICollectionGenerator());
26+
}
4327

44-
var attr = method.GetCustomAttributes(typeof(LinqExtensionMethodAttribute), false);
28+
private bool GetMethodGeneratorForType(MethodInfo method, out IHqlGeneratorForMethod methodGenerator)
29+
{
30+
methodGenerator = null;
4531

46-
if (attr.Length == 1)
47-
{
48-
// It is
49-
methodGenerator = new HqlGeneratorForExtensionMethod((LinqExtensionMethodAttribute)attr[0], method);
50-
return true;
51-
}
52-
return false;
53-
}
32+
foreach (var typeGenerator in typeGenerators.Where(typeGenerator => typeGenerator.SupportsMethod(method)))
33+
{
34+
methodGenerator = typeGenerator.GetMethodGenerator(method);
35+
return true;
36+
}
37+
return false;
38+
}
5439

55-
private bool GetRegisteredMethodGenerator(MethodInfo method, out IHqlGeneratorForMethod methodGenerator)
56-
{
57-
if (registeredMethods.TryGetValue(method, out methodGenerator))
58-
{
59-
return true;
60-
}
61-
return false;
62-
}
40+
private bool GetStandardLinqExtensionMethodGenerator(MethodInfo method, out IHqlGeneratorForMethod methodGenerator)
41+
{
42+
methodGenerator = null;
6343

64-
public IHqlGeneratorForProperty GetGenerator(MemberInfo member)
65-
{
66-
IHqlGeneratorForProperty propertyGenerator;
44+
var attr = method.GetCustomAttributes(typeof(LinqExtensionMethodAttribute), false);
6745

68-
if (registeredProperties.TryGetValue(member, out propertyGenerator))
69-
{
70-
return propertyGenerator;
71-
}
46+
if (attr.Length == 1)
47+
{
48+
// It is
49+
methodGenerator = new HqlGeneratorForExtensionMethod((LinqExtensionMethodAttribute)attr[0], method);
50+
return true;
51+
}
52+
return false;
53+
}
7254

73-
// TODO - different usage pattern to method generator
74-
return null;
75-
}
55+
private bool GetRegisteredMethodGenerator(MethodInfo method, out IHqlGeneratorForMethod methodGenerator)
56+
{
57+
if (registeredMethods.TryGetValue(method, out methodGenerator))
58+
{
59+
return true;
60+
}
61+
return false;
62+
}
7663

7764
public bool TryGetGenerator(MethodInfo method, out IHqlGeneratorForMethod generator)
7865
{
@@ -98,21 +85,21 @@ public bool TryGetGenerator(MemberInfo property, out IHqlGeneratorForProperty ge
9885
}
9986

10087
public void RegisterGenerator(MethodInfo method, IHqlGeneratorForMethod generator)
101-
{
102-
registeredMethods.Add(method, generator);
103-
}
88+
{
89+
registeredMethods.Add(method, generator);
90+
}
10491

105-
public void RegisterGenerator(MemberInfo property, IHqlGeneratorForProperty generator)
106-
{
107-
registeredProperties.Add(property, generator);
108-
}
92+
public void RegisterGenerator(MemberInfo property, IHqlGeneratorForProperty generator)
93+
{
94+
registeredProperties.Add(property, generator);
95+
}
10996

110-
private void Register(IHqlGeneratorForType typeMethodGenerator)
111-
{
112-
typeGenerators.Add(typeMethodGenerator);
113-
typeMethodGenerator.Register(this);
114-
}
115-
}
97+
private void Register(IHqlGeneratorForType typeMethodGenerator)
98+
{
99+
typeGenerators.Add(typeMethodGenerator);
100+
typeMethodGenerator.Register(this);
101+
}
102+
}
116103

117104
public class HqlGeneratorForExtensionMethod : BaseHqlGeneratorForMethod
118105
{

src/NHibernate/Linq/Visitors/SelectClauseVisitor.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -71,7 +71,7 @@ public override Expression VisitExpression(Expression expression)
7171
return base.VisitExpression(expression);
7272
}
7373

74-
private static bool CanBeEvaluatedInHqlSelectStatement(Expression expression)
74+
private bool CanBeEvaluatedInHqlSelectStatement(Expression expression)
7575
{
7676
if ((expression.NodeType == ExpressionType.MemberInit) || (expression.NodeType == ExpressionType.New) || (expression.NodeType == ExpressionType.Constant))
7777
{

0 commit comments

Comments
 (0)