Refactor: Simplify boolean conditional return statements in modularize

[LegalizeAdulthood]

Summary

Use clang-tidy to simplify conditional return statements

[alexfh]

modularize/ModularizeUtilities.cpp

389

I'd replace lines 387-393 of the original with return Extension.equals_lower(".h") || Extension.equals_lower(".inc");.

modularize/ModularizeUtilities.cpp

 // If the file extension is .h, .inc, or missing, it's
 // assumed to be a header.
 // \param FileName The file name.  Must not be a directory.
 // \returns true if it has a header extension or no extension.
 bool ModularizeUtilities::isHeader(StringRef FileName) {
   StringRef Extension = llvm::sys::path::extension(FileName);
   if (Extension.size() == 0)
     return false;
   if (Extension.equals_lower(".h"))
     return true;
-  if (Extension.equals_lower(".inc"))
-    return true;
-  return false;
+  return Extension.equals_lower(".inc");
 }
 
 // Get directory path component from file path.
 // \returns the component of the given path, which will be
 // relative if the given path is relative, absolute if the
 // given path is absolute, or "." if the path has no leading
 // path component.
 std::string ModularizeUtilities::getDirectoryFromPath(StringRef Path) {
   SmallString<256> Directory(Path);
   sys::path::remove_filename(Directory);
   if (Directory.size() == 0)
     return ".";
   return Directory.str();
 }