Thanks to visit codestin.com
Credit goes to github.com

Skip to content
This repository was archived by the owner on Dec 13, 2020. It is now read-only.

Commit 28d40c3

Browse files
committed
Cleanup, use pointer arithmetic
Inc(pointer, amount) instead of casting the pointer to an integer, then adding the offset, and casting back.
1 parent 96c5bda commit 28d40c3

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

tests/TestsFromLibGit2/t06_index.pas

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,7 @@ procedure Test06_index_read.load_an_empty_index;
5252
procedure Test06_index_read.load_a_standard_index__default_test_index_;
5353
var
5454
index: Pgit_index;
55-
i, offset: Integer;
55+
i: Integer;
5656
entries: PPgit_index_entry;
5757
e: Pgit_index_entry;
5858
begin
@@ -66,12 +66,12 @@ procedure Test06_index_read.load_a_standard_index__default_test_index_;
6666
CheckTrue(git_index_entrycount(index) = TEST_INDEX_ENTRY_COUNT);
6767
CheckTrue(index.entries.sorted = 1);
6868

69-
entries := PPgit_index_entry(index.entries.contents);
70-
7169
for i := Low(TEST_ENTRIES) to High(TEST_ENTRIES) do
7270
begin
73-
offset := TEST_ENTRIES[i].index * sizeof(Pgit_index_entry);
74-
e := PPgit_index_entry(Integer(entries) + offset)^;
71+
entries := PPgit_index_entry(index.entries.contents);
72+
73+
Inc(entries, TEST_ENTRIES[i].index);
74+
e := entries^;
7575

7676
CheckTrue(StrComp(e.path, TEST_ENTRIES[i].path) = 0);
7777
CheckTrue(e.mtime.seconds = TEST_ENTRIES[i].mtime);

0 commit comments

Comments
 (0)