Uploaded image for project: 'Core ReactOS'
  1. Core ReactOS
  2. CORE-8799

Remove explicit comparisons with TRUE.

    XMLWordPrintable

Details

    Description

      Removing some 400 "== TRUE" and "!= TRUE" and replacing with safe coding
      in every part of the system, because a BOOL has (2^32)-1 TRUE values !!!

      I've patched all those potential errors against the current trunk.
      In most cases a simple removal of "== TRUE" was sufficient, however in
      asserts I replaced it with "!= FALSE", since that may be clearer when triggered.
      The only places I let it pass was in pure debug strings and comments.

      As this is a fairly extensive patch, I would very much appreciate if a
      prioritized regression test could be run by you guys who do such things,
      since this may actually fix some "mysterious" malfunctions, or introduce
      bugs that did not trigger in my alpha test.

      My own alpha test was limited to building and installing it on VMware Player 6,
      and concluding that "it appears to run without obvious malfunctions".
      Actually, when compared to a pre-patch build, one "mysterious" crash disappeared!

      I've limited the Components and Module lists to a few of the most important ones,
      but this reaches into practically every part of the system !

      Best Regards
      // Love

      Attachments

        1. Remove_explicit_eq_TRUE.patch
          142 kB
          LoveN
        2. TRUE_FALSE_no_Wine.patch
          109 kB
          LoveN
        3. TRUE_FALSE.patch
          110 kB
          Timo Kreuzer

        Issue Links

          Activity

            People

              bug zilla Bug Zilla
              LoveN LoveN
              Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: