Skip to content

Commit

Permalink
Don't add the FIXME for converted macros
Browse files Browse the repository at this point in the history
We're pretty sure this is correct and anyone who runs this script is likely to have source control to see the differences
  • Loading branch information
slouken committed Jan 24, 2024
1 parent 006edca commit b8c4ea3
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion build-scripts/rename_macros.py
Original file line number Diff line number Diff line change
Expand Up @@ -63,7 +63,7 @@ def cb(m):
original = m.group(0)
match_start, _ = m.span(0)
platform_start, platform_end = m.span(1)
new_text = "{0} /* FIXME: use '#ifdef {0}' or 'defined({0})' */".format(self.RENAMED_MACROS[macro])
new_text = self.RENAMED_MACROS[macro]
r = original[:(platform_start-match_start)] + new_text + original[platform_end-match_start:]
return r
contents, _ = self.re_platform_macros.subn(cb, contents)
Expand Down

0 comments on commit b8c4ea3

Please sign in to comment.