Apply suggestions from code review

dotnet · Nov 3, 2021 · 970a4a8 · 970a4a8
1 parent a5ca169
commit 970a4a8
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/...xt.RegularExpressions/src/System/Text/RegularExpressions/Symbolic/SymbolicRegexMatcher.cs b/...xt.RegularExpressions/src/System/Text/RegularExpressions/Symbolic/SymbolicRegexMatcher.cs
@@ -359,8 +359,7 @@ public DfaMatchingState<TSetType> TakeTransition(
                 Debug.Assert(builder._delta is not null);
 
                 int offset = (currentState.Id << builder._mintermsCount) | mintermId;
-                DfaMatchingState<TSetType>? p = Volatile.Read(ref builder._delta[offset]);
-                return p ?? matcher.CreateNewTransition(currentState, minterm, offset);
+                return Volatile.Read(ref builder._delta[offset]) ?? matcher.CreateNewTransition(currentState, minterm, offset);
             }
         }
 
@@ -390,8 +389,7 @@ public DfaMatchingState<TSetType> TakeTransition(
                     DfaMatchingState<TSetType> nextStates = builder.MkState(oneState, currentStates.PrevCharKind);
 
                     int offset = (nextStates.Id << builder._mintermsCount) | mintermId;
-                    DfaMatchingState<TSetType>? p_ = Volatile.Read(ref builder._delta[offset]);
-                    DfaMatchingState<TSetType> p = p_ ?? matcher.CreateNewTransition(nextStates, minterm, offset);
+                    DfaMatchingState<TSetType> p = Volatile.Read(ref builder._delta[offset]) ?? matcher.CreateNewTransition(nextStates, minterm, offset);
 
                     // Observe that if p.Node is an Or it will be flattened.
                     union = builder.MkOr2(union, p.Node);