Page MenuHomePhorge

Make sure `offset` is an integer
ClosedPublic

Authored by avivey on Apr 12 2024, 08:32.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Jul 12, 17:17
Unknown Object (File)
Tue, Jul 9, 01:44
Unknown Object (File)
Mon, Jul 8, 02:47
Unknown Object (File)
Wed, Jun 26, 00:42
Unknown Object (File)
Mon, Jun 24, 10:50
Unknown Object (File)
Mon, Jun 24, 02:33
Unknown Object (File)
Sun, Jun 23, 12:57
Unknown Object (File)
Fri, Jun 21, 14:09

Details

Summary

Older git versions allowed the --skip to get empty string as value and interpreted that to an int.
We were sending null in some places where we wanted a 0, so it all worked out.

Since https://git.kernel.org/pub/scm/git/git.git/commit/revision.c?id=71a1e94821666909b7b2bd62a36244c601f8430e git fails when provided empty string for an integer.

See Q124 and T15783.

Test Plan

Have recent git version (that fails with --skip '', navigate to /diffusion/1/history/master/) - see history.

Diff Detail

Repository
rP Phorge
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

avivey requested review of this revision.Apr 12 2024, 08:32

Where I can document myself about undefined_magic_text? Undetected with grep 🤔

Where I can document myself about undefined_magic_text? Undetected with grep 🤔

??

What for?

(Stupid question :D sorry - clear now)

sgtm

This revision is now accepted and ready to land.Apr 12 2024, 13:56
aklapper subscribed.

(I took the liberty to link the specific change in Git in the commit message)

This revision was automatically updated to reflect the committed changes.