[PATCH] D141611: [include-mapping] Print an error message in case the symbol index points to a non-existent page.

Kadir Cetinkaya via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jan 12 07:50:10 PST 2023


kadircet accepted this revision.
kadircet added a comment.
This revision is now accepted and ready to land.

thanks!



================
Comment at: clang/tools/include-mapping/cppreference_parser.py:149
+      else:
+        sys.stderr.write("Could not parse page: %s. Page does not exist.\n" % path)                
 
----------------
i think it's useful to mention the symbol name as well, maybe something like:

`sys.stderr.write("Discarding information for '%s', page '%s' doesn't exist." % (symbol_name, path))`


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D141611/new/

https://reviews.llvm.org/D141611



More information about the cfe-commits mailing list