* [PATCH] git: use signed counters when comparing with signed vars
@ 2024-12-18 0:15 AreaZR via GitGitGadget
2024-12-18 1:35 ` [PATCH v2] git: use signed loop " AreaZR via GitGitGadget
0 siblings, 1 reply; 3+ messages in thread
From: AreaZR via GitGitGadget @ 2024-12-18 0:15 UTC (permalink / raw)
To: git; +Cc: AreaZR, Seija Kijin
From: Seija Kijin <doremylover123@gmail.com>
The compiler (at least LLVM) treats them better anyway
because of the C standard not defining signed overflow.
Signed-off-by: Seija Kijin <doremylover123@gmail.com>
---
git: use signed counters when comparing with signed vars
The compiler (at least LLVM) treats them better anyway because of the C
standard not defining signed overflow.
Published-As: https://github.com/gitgitgadget/git/releases/tag/pr-git-1843%2FAreaZR%2Fone-more-time-v1
Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-git-1843/AreaZR/one-more-time-v1
Pull-Request: https://github.com/git/git/pull/1843
add-interactive.c | 2 +-
bloom.c | 2 +-
git.c | 2 +-
3 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/add-interactive.c b/add-interactive.c
index 49042b30261..12c23fa10ba 100644
--- a/add-interactive.c
+++ b/add-interactive.c
@@ -848,7 +848,7 @@ static int get_untracked_files(struct repository *r,
const struct pathspec *ps)
{
struct dir_struct dir = { 0 };
- size_t i;
+ int i;
struct strbuf buf = STRBUF_INIT;
if (repo_read_index(r) < 0)
diff --git a/bloom.c b/bloom.c
index c4286341059..42db509bea9 100644
--- a/bloom.c
+++ b/bloom.c
@@ -314,7 +314,7 @@ static int has_entries_with_high_bit(struct repository *r, struct tree *t)
init_tree_desc(&desc, &t->object.oid, t->buffer, t->size);
while (tree_entry(&desc, &entry)) {
- size_t i;
+ int i;
for (i = 0; i < entry.pathlen; i++) {
if (entry.path[i] & 0x80) {
t->object.flags |= HIGH_BITS;
diff --git a/git.c b/git.c
index 46b3c740c5d..86f7112f4bf 100644
--- a/git.c
+++ b/git.c
@@ -946,7 +946,7 @@ int cmd_main(int argc, const char **argv)
*/
setup_path();
- for (size_t i = 0; i < argc; i++)
+ for (int i = 0; i < argc; i++)
strvec_push(&args, argv[i]);
while (1) {
base-commit: 2ccc89b0c16c51561da90d21cfbb4b58cc877bf6
--
gitgitgadget
^ permalink raw reply related [flat|nested] 3+ messages in thread
* [PATCH v2] git: use signed loop counters when comparing with signed vars
2024-12-18 0:15 [PATCH] git: use signed counters when comparing with signed vars AreaZR via GitGitGadget
@ 2024-12-18 1:35 ` AreaZR via GitGitGadget
2024-12-18 3:10 ` [PATCH v3] " AreaZR via GitGitGadget
0 siblings, 1 reply; 3+ messages in thread
From: AreaZR via GitGitGadget @ 2024-12-18 1:35 UTC (permalink / raw)
To: git; +Cc: AreaZR, Seija Kijin
From: Seija Kijin <doremylover123@gmail.com>
The compiler (at least LLVM) treats them better anyway
because of the C standard not defining signed overflow.
Signed-off-by: Seija Kijin <doremylover123@gmail.com>
---
git: use signed loop counters when comparing with signed vars
The compiler (at least LLVM) treats them better anyway because of the C
standard not defining signed overflow.
Published-As: https://github.com/gitgitgadget/git/releases/tag/pr-git-1843%2FAreaZR%2Fone-more-time-v2
Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-git-1843/AreaZR/one-more-time-v2
Pull-Request: https://github.com/git/git/pull/1843
Range-diff vs v1:
1: 1dea0ce98b4 ! 1: c8552548fe9 git: use signed counters when comparing with signed vars
@@ Metadata
Author: Seija Kijin <doremylover123@gmail.com>
## Commit message ##
- git: use signed counters when comparing with signed vars
+ git: use signed loop counters when comparing with signed vars
The compiler (at least LLVM) treats them better anyway
because of the C standard not defining signed overflow.
add-interactive.c | 2 +-
bloom.c | 2 +-
git.c | 2 +-
3 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/add-interactive.c b/add-interactive.c
index 49042b30261..12c23fa10ba 100644
--- a/add-interactive.c
+++ b/add-interactive.c
@@ -848,7 +848,7 @@ static int get_untracked_files(struct repository *r,
const struct pathspec *ps)
{
struct dir_struct dir = { 0 };
- size_t i;
+ int i;
struct strbuf buf = STRBUF_INIT;
if (repo_read_index(r) < 0)
diff --git a/bloom.c b/bloom.c
index c4286341059..42db509bea9 100644
--- a/bloom.c
+++ b/bloom.c
@@ -314,7 +314,7 @@ static int has_entries_with_high_bit(struct repository *r, struct tree *t)
init_tree_desc(&desc, &t->object.oid, t->buffer, t->size);
while (tree_entry(&desc, &entry)) {
- size_t i;
+ int i;
for (i = 0; i < entry.pathlen; i++) {
if (entry.path[i] & 0x80) {
t->object.flags |= HIGH_BITS;
diff --git a/git.c b/git.c
index 46b3c740c5d..86f7112f4bf 100644
--- a/git.c
+++ b/git.c
@@ -946,7 +946,7 @@ int cmd_main(int argc, const char **argv)
*/
setup_path();
- for (size_t i = 0; i < argc; i++)
+ for (int i = 0; i < argc; i++)
strvec_push(&args, argv[i]);
while (1) {
base-commit: 063bcebf0c917140ca0e705cbe0fdea127e90086
--
gitgitgadget
^ permalink raw reply related [flat|nested] 3+ messages in thread
* [PATCH v3] git: use signed loop counters when comparing with signed vars
2024-12-18 1:35 ` [PATCH v2] git: use signed loop " AreaZR via GitGitGadget
@ 2024-12-18 3:10 ` AreaZR via GitGitGadget
0 siblings, 0 replies; 3+ messages in thread
From: AreaZR via GitGitGadget @ 2024-12-18 3:10 UTC (permalink / raw)
To: git; +Cc: AreaZR, Seija Kijin
From: Seija Kijin <doremylover123@gmail.com>
The compiler (at least LLVM) treats them better anyway
because of the C standard not defining signed overflow.
Signed-off-by: Seija Kijin <doremylover123@gmail.com>
---
git: use signed loop counters when comparing with signed vars
The compiler (at least LLVM) treats them better anyway because of the C
standard not defining signed overflow.
Published-As: https://github.com/gitgitgadget/git/releases/tag/pr-git-1843%2FAreaZR%2Fone-more-time-v3
Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-git-1843/AreaZR/one-more-time-v3
Pull-Request: https://github.com/git/git/pull/1843
Range-diff vs v2:
1: c8552548fe9 = 1: 754e4a60ff6 git: use signed loop counters when comparing with signed vars
add-interactive.c | 2 +-
bloom.c | 2 +-
git.c | 2 +-
3 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/add-interactive.c b/add-interactive.c
index 49042b30261..12c23fa10ba 100644
--- a/add-interactive.c
+++ b/add-interactive.c
@@ -848,7 +848,7 @@ static int get_untracked_files(struct repository *r,
const struct pathspec *ps)
{
struct dir_struct dir = { 0 };
- size_t i;
+ int i;
struct strbuf buf = STRBUF_INIT;
if (repo_read_index(r) < 0)
diff --git a/bloom.c b/bloom.c
index c4286341059..42db509bea9 100644
--- a/bloom.c
+++ b/bloom.c
@@ -314,7 +314,7 @@ static int has_entries_with_high_bit(struct repository *r, struct tree *t)
init_tree_desc(&desc, &t->object.oid, t->buffer, t->size);
while (tree_entry(&desc, &entry)) {
- size_t i;
+ int i;
for (i = 0; i < entry.pathlen; i++) {
if (entry.path[i] & 0x80) {
t->object.flags |= HIGH_BITS;
diff --git a/git.c b/git.c
index 46b3c740c5d..86f7112f4bf 100644
--- a/git.c
+++ b/git.c
@@ -946,7 +946,7 @@ int cmd_main(int argc, const char **argv)
*/
setup_path();
- for (size_t i = 0; i < argc; i++)
+ for (int i = 0; i < argc; i++)
strvec_push(&args, argv[i]);
while (1) {
base-commit: d882f382b3d939d90cfa58d17b17802338f05d66
--
gitgitgadget
^ permalink raw reply related [flat|nested] 3+ messages in thread
end of thread, other threads:[~2024-12-18 3:10 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-12-18 0:15 [PATCH] git: use signed counters when comparing with signed vars AreaZR via GitGitGadget
2024-12-18 1:35 ` [PATCH v2] git: use signed loop " AreaZR via GitGitGadget
2024-12-18 3:10 ` [PATCH v3] " AreaZR via GitGitGadget
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).