[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[csmith-project/creduce] 41a65e: Fix condition logic in RenameClass::isValidName



  Branch: refs/heads/master
  Home:   https://github.com/csmith-project/creduce
  Commit: 41a65e365fe86954ec65ff1e6836bf0319e28e25
      https://github.com/csmith-project/creduce/commit/41a65e365fe86954ec65ff1e6836bf0319e28e25
  Author: Boris Egorov <egorov@linux.com>
  Date:   2016-03-30 (Wed, 30 Mar 2016)

  Changed paths:
    M clang_delta/RenameClass.cpp

  Log Message:
  -----------
  Fix condition logic in RenameClass::isValidName

Detected by GCC's "logical-op" check:

    RenameClass.cpp: In member function ‘bool RenameClass::isValidName(const string&)’:
    RenameClass.cpp:163:23: warning: logical ‘or’ of collectively exhaustive tests is always true [-Wlogical-op]
  return (((C >= 'A') || (C <= 'Z')) && !isReservedName(C));
           ~~~~~~~~~~~^~~~~~~~~~~~~


  Commit: 445f98a46ce5273262139eef372ce3b11bae9b33
      https://github.com/csmith-project/creduce/commit/445f98a46ce5273262139eef372ce3b11bae9b33
  Author: Yang Chen <chenyang@cs.utah.edu>
  Date:   2016-03-31 (Thu, 31 Mar 2016)

  Changed paths:
    M clang_delta/RenameClass.cpp

  Log Message:
  -----------
  Merge pull request #91 from JIghtuse/master

Fix condition logic in RenameClass::isValidName


Compare: https://github.com/csmith-project/creduce/compare/b9d81d33a587...445f98a46ce5