dir: avoid -Wdiscarded-qualifiers in remove_path()
When building with glibc-2.43 there is the following warning: dir.c:3526:15: warning: assignment discards ‘const’ qualifier from pointer target type [-Wdiscarded-qualifiers] 3526 | slash = strrchr(name, '/'); | ^ In this case we use a non-const pointer to get the last slash of the unwritable file name, and then use it again to write in the strdup'd file name. We can avoid this warning and make the code a bit more clear by using a separate variable to access the original argument and its strdup'd copy. Signed-off-by: Collin Funk <collin.funk1@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com>
Collin Funk committed
Mar 8, 2026 at 20:23 UTC
02cbae61df7b3493e7f76f8385fc9257de60755f
1 file changed
+4
-4
dir.c
+4
-4
@@ -3516,15 +3516,15 @@ int get_sparse_checkout_patterns(struct pattern_list *pl)
3516
3517
int remove_path(const char *name)
3518
{
3519
- char *slash;
3519
+ const char *last;
3520
3521
if (unlink(name) && !is_missing_file_error(errno))
3522
return -1;
3523
3524
- slash = strrchr(name, '/');
3525
- if (slash) {
3524
+ last = strrchr(name, '/');
3525
+ if (last) {
3526
char *dirs = xstrdup(name);
3527
- slash = dirs + (slash - name);
3527
+ char *slash = dirs + (last - name);
3528
do {
3529
*slash = '\0';
3530
if (startup_info->original_cwd &&