diff options
| author | Elijah Newren <newren@gmail.com> | 2024-06-19 03:00:15 +0000 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2024-06-20 10:35:24 -0700 |
| commit | 9ed8e17d8a8c374529bb908c81f1a862f689b904 (patch) | |
| tree | d82abc4238274aac4eb5474b8674e9342fc7acd9 /git-send-email.perl | |
| parent | merge-ort: maintain expected invariant for priv member (diff) | |
| download | git-9ed8e17d8a8c374529bb908c81f1a862f689b904.tar.gz git-9ed8e17d8a8c374529bb908c81f1a862f689b904.zip | |
merge-ort: fix type of local 'clean' var in handle_content_merge ()
handle_content_merge() returns an int. Every caller of
handle_content_merge() expects an int. However, we declare a local
variable 'clean' that we use for the return value to be unsigned. To
make matters worse, we also assign 'clean' the return value of
merge_submodule() in one codepath, which is defined to return an int.
It seems that the only reason to have 'clean' be unsigned was to allow a
cutesy bit manipulation operation to be well-defined. Fix the type of
the 'clean' local in handle_content_merge().
Signed-off-by: Elijah Newren <newren@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'git-send-email.perl')
0 files changed, 0 insertions, 0 deletions
