https://bugs.documentfoundation.org/show_bug.cgi?id=96505

            Bug ID: 96505
           Summary: Get rid of cargo cult "long" integer literals
           Product: LibreOffice
           Version: unspecified
          Hardware: All
                OS: All
            Status: UNCONFIRMED
          Keywords: difficultyBeginner, easyHack, skillCpp
          Severity: normal
          Priority: medium
         Component: LibreOffice
          Assignee: [email protected]
          Reporter: [email protected]

We have tons of instances of the 'long' integer literals 0L and 1L in the code.
(And also some 2L etc.) I am fairly certain that in the majority of cases, no
compiler warnings would be produced, and the semantics of the code would be
identical, and the generated machine code would be identical, if the 'L' was
dropped. But feel free to prove me wrong, of course.

If you want to work on this, don't change them all in one go. Do it a
"reasonable" number of source files at a time. (So that the total number of
changed lines in each change is a hundred or so.) And make sure that you indeed
don't introduce any compiler warnings. Don't do a change if you are at all
unsure whether it indeed might have a semantic effect on what the code means.

-- 
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Libreoffice-bugs mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

Reply via email to