Skip to content

Commit 0201d46

Browse files
committed
Avoid complicated arithmetics API for basic operation
1 parent cf82f3c commit 0201d46

1 file changed

Lines changed: 4 additions & 4 deletions

File tree

Orm/Xtensive.Orm/Orm/Internals/TupleExtensions.cs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ public static bool ContainsNonEmptyValues(this Tuple target)
3535

3636
public static bool ContainsEmptyValues(this Tuple target, in Segment<int> segment)
3737
{
38-
for (int i = segment.Offset; i < segment.EndOffset; i++) {
38+
for (int i = segment.Offset, endOffset = segment.Offset + segment.Length; i < endOffset; i++) {
3939
var state = target.GetFieldState(i);
4040
if (!state.HasValue())
4141
return true;
@@ -45,7 +45,7 @@ public static bool ContainsEmptyValues(this Tuple target, in Segment<int> segmen
4545

4646
public static bool ContainsNonEmptyValues(this Tuple target, in Segment<int> segment)
4747
{
48-
for (int i = segment.Offset; i < segment.EndOffset; i++) {
48+
for (int i = segment.Offset, endOffset = segment.Offset + segment.Length; i < endOffset; i++) {
4949
var state = target.GetFieldState(i);
5050
if (state.HasValue())
5151
return true;
@@ -55,7 +55,7 @@ public static bool ContainsNonEmptyValues(this Tuple target, in Segment<int> seg
5555

5656
public static bool AreAllColumnsAvalilable(this Tuple target, in Segment<int> segment)
5757
{
58-
for (int i = segment.Offset; i < segment.EndOffset; i++) {
58+
for (int i = segment.Offset, endOffset = segment.Offset + segment.Length; i < endOffset; i++) {
5959
var state = target.GetFieldState(i);
6060
if (!state.IsAvailable())
6161
return false;
@@ -65,7 +65,7 @@ public static bool AreAllColumnsAvalilable(this Tuple target, in Segment<int> se
6565

6666
public static bool IsAtLeastOneColumAvailable(this Tuple target, in Segment<int> segment)
6767
{
68-
for (int i = segment.Offset; i < segment.EndOffset; i++) {
68+
for (int i = segment.Offset, endOffset = segment.Offset + segment.Length; i < endOffset; i++) {
6969
var state = target.GetFieldState(i);
7070
if (state.IsAvailable())
7171
return true;

0 commit comments

Comments
 (0)