Skip to content

Commit 3988a21

Browse files
Simplify fallback
1 parent 53cdcfc commit 3988a21

File tree

1 file changed

+9
-15
lines changed

1 file changed

+9
-15
lines changed

src/NHibernate/Bytecode/Lightweight/ReflectionOptimizer.cs

Lines changed: 9 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,8 @@
11
using System;
2-
using System.Collections.Generic;
32
using System.Reflection;
43
using System.Reflection.Emit;
54
using System.Security;
65
using System.Security.Permissions;
7-
using NHibernate.Linq;
86
using NHibernate.Properties;
97
using NHibernate.Util;
108

@@ -44,8 +42,9 @@ public ReflectionOptimizer(System.Type mappedType, IGetter[] getters, ISetter[]
4442
/// <summary>
4543
/// Class constructor.
4644
/// </summary>
47-
public ReflectionOptimizer(System.Type mappedType, IGetter[] getters, ISetter[] setters,
48-
IGetter specializedGetter, ISetter specializedSetter)
45+
public ReflectionOptimizer(
46+
System.Type mappedType, IGetter[] getters, ISetter[] setters,
47+
IGetter specializedGetter, ISetter specializedSetter)
4948
{
5049
// save off references
5150
this.mappedType = mappedType;
@@ -59,27 +58,22 @@ public ReflectionOptimizer(System.Type mappedType, IGetter[] getters, ISetter[]
5958
var getMethods = new GetPropertyValueInvoker[getters.Length];
6059
for (var i = 0; i < getters.Length; i++)
6160
{
62-
getMethods[i] = GenerateGetPropertyValueMethod(getters[i]) ?? getters[i].Get;
61+
getMethods[i] = GenerateGetPropertyValueMethod(getters[i]);
6362
}
6463

6564
var setMethods = new SetPropertyValueInvoker[setters.Length];
6665
for (var i = 0; i < setters.Length; i++)
6766
{
68-
setMethods[i] = GenerateSetPropertyValueMethod(setters[i]) ?? setters[i].Set;
67+
setMethods[i] = GenerateSetPropertyValueMethod(setters[i]);
6968
}
7069

7170
accessOptimizer = new AccessOptimizer(
7271
getInvoker,
7372
setInvoker,
7473
getMethods,
7574
setMethods,
76-
// 6.0 TODO: Remove ternary ifs once the obsolete constructor is removed
77-
specializedGetter != null
78-
? GenerateGetPropertyValueMethod(specializedGetter) ?? specializedGetter.Get
79-
: null,
80-
specializedSetter != null
81-
? GenerateSetPropertyValueMethod(specializedSetter) ?? specializedSetter.Set
82-
: null
75+
GenerateGetPropertyValueMethod(specializedGetter),
76+
GenerateSetPropertyValueMethod(specializedSetter)
8377
);
8478

8579
createInstanceMethod = CreateCreateInstanceMethod(mappedType);
@@ -171,7 +165,7 @@ private GetPropertyValueInvoker GenerateGetPropertyValueMethod(IGetter getter)
171165
{
172166
if (!(getter is IOptimizableGetter optimizableGetter))
173167
{
174-
return null;
168+
return getter.Get;
175169
}
176170

177171
var method = CreateDynamicMethod(typeof(object), new[] { typeof(object) });
@@ -191,7 +185,7 @@ private SetPropertyValueInvoker GenerateSetPropertyValueMethod(ISetter setter)
191185
{
192186
if (!(setter is IOptimizableSetter optimizableSetter))
193187
{
194-
return null;
188+
return setter.Set;
195189
}
196190

197191
// void (target, value) { ((ClassType) target).SetMethod((FieldType) value); }

0 commit comments

Comments
 (0)