File tree Expand file tree Collapse file tree 1 file changed +0
-33
lines changed
Expand file tree Collapse file tree 1 file changed +0
-33
lines changed Original file line number Diff line number Diff line change @@ -12,39 +12,6 @@ There are some suggestions on how you can find some microprojects on your own in
1212
1313## Ideas for microprojects
1414
15- ### Fix Sign Comparison Warnings in Git's Codebase
16-
17- Help improve Git's code quality by fixing sign comparison warnings in files that
18- currently disable these warnings. The goal is to remove instances of
19- ` DISABLE_SIGN_COMPARE_WARNINGS ` macro and fix the underlying issues properly.
20-
21- #### Steps to Complete
22- 1 . Find a C source file that contains ` #define DISABLE_SIGN_COMPARE_WARNINGS `
23- 2 . Remove this #define
24- 3 . Build Git with ` DEVELOPER=1 ` to enable compiler warnings. The ` DEVLEOPER `
25- can be specified in your ` config.mak ` or as follows
26-
27- ``` sh
28- make DEVELOPER=1 -j4
29- ```
30-
31- 4 . Fix all ` -Wsign-compare ` warnings that appear for that file:
32- - Pay attention to comparisons between signed and unsigned integers
33- - Modify variable types or add appropriate casts as needed
34- - Ensure the fixes don't change the code's behavior
35-
36- #### Notes
37- - Each file should be handled in a separate patch
38- - Follow Git's commit message conventions
39- - Test your changes thoroughly
40- - This is part of an ongoing effort to enable ` -Wsign-compare ` globally
41-
42- #### Related Patches
43- For context on why this is a crucial improvement to Git's codebase, checkout
44- [ this e-mail
] ( https://public-inbox.org/git/[email protected] / ) 45- by Patrick Steinhardt.
46-
47-
4815### Modernize Test Path Checking in Git's Test Suite
4916
5017Help improve Git's test suite by converting old-style path checks to use modern
You can’t perform that action at this time.
0 commit comments