יום חמישי, ספטמבר 05, 2024

תזהרו משינוי ההקשר בgitk מ 3 שורות לשורה אחת, זה מטעה בטרוף

טוב טעויות של חוסר ריכוז בהגדרות יכולות לעלות הרבה, והנה דוגמא כזאת שבגללה שרפתי יותר משעה עבודה היום.
.
ב gitk  יש דרך לשנות את ההקשר (מספר השורות שמוצגות), מכיוון שהייתי קצת עצלן שיניתי את ההקשר לשורה אחת, הרי גם ככה אני עושה git add -p בשביל לברור קומיטים ברמת השורה, אז קל יותר לראות את הינויים.

היה לי איזה באג שהיתי צריך לתקן, והפעלתי gitk עם שורה אחת, הפלט שראיתי היה : 
index 4e92ce88879..be1c749241f 100644
--- a/x/y/filesystem/remote_fs.cpp
+++ b/x/y/filesystem/remote_fs.cpp
@@ -10219,7 +10219,7 @@ file_state_t __update_inode(
-          permission_t * &         r_permission,
+    const permission_t * &         r_permission,
אני מסתקל בקוד ,מתקן ועדיין יש חתיכת תקלה, לאחר 20 דקות שאני שובר את הראש אני פונה לעמיתי, יש לנו כלל בחברה, שאם אתה תקוע מעל 20 דקות על משהוא אתה חייב לפנות לעוד זוג עיניים, ולא הברווז לדיבוג זה לא שווה.
 
הוא מסתקל, ואומר , רגע אם אני מבצע checkout לשינוי הזה הפונקציה הזאת לא השתנתה כלל ! 
 
 אז אני מפעיל git show שבו יש הקשר גדול יותר, ומה אני רואה ? 

index 4e92ce88879..be1c749241f 100644
--- a/x/y/filesystem/remote_fs.cpp
+++ b/x/y/filesystem/remote_fs.cpp
@@ -10219,7 +10219,7 @@ file_state_t __update_inode(
  file_state_t  __create_link(
     const   char         * pt_calling_source_file,
     const   unsigned int   calling_line_number,
-          permission_t * &         r_permission,
+    const permission_t * &         r_permission,

למרות שהפונקציה שהשתנתה היא __create_link הוא היה מציג את הפונקציה __update_inode.

אין תגובות: