diff options
| author | Rob Mensching <rob@firegiant.com> | 2019-10-18 11:28:14 -0700 |
|---|---|---|
| committer | Rob Mensching <rob@firegiant.com> | 2019-10-18 11:29:59 -0700 |
| commit | cf6531a216dc248177116399120f02d67731f312 (patch) | |
| tree | f09e4dd37dcd399758fdb82e23f0eeeb1f9e528e /src/WixToolset.Data/IntermediateTupleExtensions.cs | |
| parent | c5bae04ebcf3cae706ca3bc2717ff3d59ec04824 (diff) | |
| download | wix-cf6531a216dc248177116399120f02d67731f312.tar.gz wix-cf6531a216dc248177116399120f02d67731f312.tar.bz2 wix-cf6531a216dc248177116399120f02d67731f312.zip | |
Small fixes to tuples
Diffstat (limited to 'src/WixToolset.Data/IntermediateTupleExtensions.cs')
| -rw-r--r-- | src/WixToolset.Data/IntermediateTupleExtensions.cs | 25 |
1 files changed, 5 insertions, 20 deletions
diff --git a/src/WixToolset.Data/IntermediateTupleExtensions.cs b/src/WixToolset.Data/IntermediateTupleExtensions.cs index dd488dc5..7fb2a6c3 100644 --- a/src/WixToolset.Data/IntermediateTupleExtensions.cs +++ b/src/WixToolset.Data/IntermediateTupleExtensions.cs | |||
| @@ -16,11 +16,6 @@ namespace WixToolset.Data | |||
| 16 | 16 | ||
| 17 | public static IntermediateField Set(this IntermediateTuple tuple, int index, bool value) | 17 | public static IntermediateField Set(this IntermediateTuple tuple, int index, bool value) |
| 18 | { | 18 | { |
| 19 | if (value == default && NoFieldMetadata(tuple, index)) | ||
| 20 | { | ||
| 21 | return tuple.Fields[index] = null; | ||
| 22 | } | ||
| 23 | |||
| 24 | var definition = tuple.Definition.FieldDefinitions[index]; | 19 | var definition = tuple.Definition.FieldDefinitions[index]; |
| 25 | 20 | ||
| 26 | var field = tuple.Fields[index].Set(definition, value); | 21 | var field = tuple.Fields[index].Set(definition, value); |
| @@ -30,7 +25,7 @@ namespace WixToolset.Data | |||
| 30 | 25 | ||
| 31 | public static IntermediateField Set(this IntermediateTuple tuple, int index, bool? value) | 26 | public static IntermediateField Set(this IntermediateTuple tuple, int index, bool? value) |
| 32 | { | 27 | { |
| 33 | if (value == default && NoFieldMetadata(tuple, index)) | 28 | if (value == null && NoFieldMetadata(tuple, index)) |
| 34 | { | 29 | { |
| 35 | return tuple.Fields[index] = null; | 30 | return tuple.Fields[index] = null; |
| 36 | } | 31 | } |
| @@ -44,11 +39,6 @@ namespace WixToolset.Data | |||
| 44 | 39 | ||
| 45 | public static IntermediateField Set(this IntermediateTuple tuple, int index, long value) | 40 | public static IntermediateField Set(this IntermediateTuple tuple, int index, long value) |
| 46 | { | 41 | { |
| 47 | if (value == default && NoFieldMetadata(tuple, index)) | ||
| 48 | { | ||
| 49 | return tuple.Fields[index] = null; | ||
| 50 | } | ||
| 51 | |||
| 52 | var definition = tuple.Definition.FieldDefinitions[index]; | 42 | var definition = tuple.Definition.FieldDefinitions[index]; |
| 53 | 43 | ||
| 54 | var field = tuple.Fields[index].Set(definition, value); | 44 | var field = tuple.Fields[index].Set(definition, value); |
| @@ -58,7 +48,7 @@ namespace WixToolset.Data | |||
| 58 | 48 | ||
| 59 | public static IntermediateField Set(this IntermediateTuple tuple, int index, long? value) | 49 | public static IntermediateField Set(this IntermediateTuple tuple, int index, long? value) |
| 60 | { | 50 | { |
| 61 | if (value == default && NoFieldMetadata(tuple, index)) | 51 | if (value == null && NoFieldMetadata(tuple, index)) |
| 62 | { | 52 | { |
| 63 | return tuple.Fields[index] = null; | 53 | return tuple.Fields[index] = null; |
| 64 | } | 54 | } |
| @@ -72,11 +62,6 @@ namespace WixToolset.Data | |||
| 72 | 62 | ||
| 73 | public static IntermediateField Set(this IntermediateTuple tuple, int index, int value) | 63 | public static IntermediateField Set(this IntermediateTuple tuple, int index, int value) |
| 74 | { | 64 | { |
| 75 | if (value == default && NoFieldMetadata(tuple, index)) | ||
| 76 | { | ||
| 77 | return tuple.Fields[index] = null; | ||
| 78 | } | ||
| 79 | |||
| 80 | var definition = tuple.Definition.FieldDefinitions[index]; | 65 | var definition = tuple.Definition.FieldDefinitions[index]; |
| 81 | 66 | ||
| 82 | var field = tuple.Fields[index].Set(definition, value); | 67 | var field = tuple.Fields[index].Set(definition, value); |
| @@ -86,7 +71,7 @@ namespace WixToolset.Data | |||
| 86 | 71 | ||
| 87 | public static IntermediateField Set(this IntermediateTuple tuple, int index, int? value) | 72 | public static IntermediateField Set(this IntermediateTuple tuple, int index, int? value) |
| 88 | { | 73 | { |
| 89 | if (value == default && NoFieldMetadata(tuple, index)) | 74 | if (value == null && NoFieldMetadata(tuple, index)) |
| 90 | { | 75 | { |
| 91 | return tuple.Fields[index] = null; | 76 | return tuple.Fields[index] = null; |
| 92 | } | 77 | } |
| @@ -100,7 +85,7 @@ namespace WixToolset.Data | |||
| 100 | 85 | ||
| 101 | public static IntermediateField Set(this IntermediateTuple tuple, int index, IntermediateFieldPathValue value) | 86 | public static IntermediateField Set(this IntermediateTuple tuple, int index, IntermediateFieldPathValue value) |
| 102 | { | 87 | { |
| 103 | if (value == default && NoFieldMetadata(tuple, index)) | 88 | if (value == null && NoFieldMetadata(tuple, index)) |
| 104 | { | 89 | { |
| 105 | return tuple.Fields[index] = null; | 90 | return tuple.Fields[index] = null; |
| 106 | } | 91 | } |
| @@ -114,7 +99,7 @@ namespace WixToolset.Data | |||
| 114 | 99 | ||
| 115 | public static IntermediateField Set(this IntermediateTuple tuple, int index, string value) | 100 | public static IntermediateField Set(this IntermediateTuple tuple, int index, string value) |
| 116 | { | 101 | { |
| 117 | if (value == default && NoFieldMetadata(tuple, index)) | 102 | if (value == null && NoFieldMetadata(tuple, index)) |
| 118 | { | 103 | { |
| 119 | return tuple.Fields[index] = null; | 104 | return tuple.Fields[index] = null; |
| 120 | } | 105 | } |
