[clang-tidy] Fix add_new_check.py.

[curdeius]

Summary

This fixes the case where the check should be added at the end of the list of checks in CMakeLists.txt of the corresponding module.
Before it was added after LINK_LIBS line.

[alexfh]

Looks good. Thank you for the fix!

[curdeius]

Can you land it please?
I don't have write access.

Thanks in advance!

Marek Kurdej

[alexfh]

Can you land it please?

Sure. Committed revision 246854.

clang-tidy/add_new_check.py

def adapt_cmake(module_path, check_name_camel):
   for line in lines:
     if line.strip() == cpp_file:
       return False
 
   with open(filename, 'w') as f:
     cpp_found = False
     file_added = False
     for line in lines:
-      if not file_added and (line.endswith('.cpp') or cpp_found):
+      cpp_line = line.endswith('.cpp')
+      if (not file_added) and (cpp_line or cpp_found):
         cpp_found = True
-        if line.strip() > cpp_file:
+        if (line.strip() > cpp_file) or (not cpp_line):
           f.write('  ' + cpp_file + '\n')
           file_added = True
       f.write(line + '\n')
 
   return True
 
 
 # Adds a header for the new check.