check_stdlib_usage.py 6.6 KB

123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261
  1. #!/usr/bin/env python3
  2. #
  3. # Simple DirectMedia Layer
  4. # Copyright (C) 1997-2025 Sam Lantinga <[email protected]>
  5. #
  6. # This software is provided 'as-is', without any express or implied
  7. # warranty. In no event will the authors be held liable for any damages
  8. # arising from the use of this software.
  9. #
  10. # Permission is granted to anyone to use this software for any purpose,
  11. # including commercial applications, and to alter it and redistribute it
  12. # freely, subject to the following restrictions:
  13. #
  14. # 1. The origin of this software must not be misrepresented; you must not
  15. # claim that you wrote the original software. If you use this software
  16. # in a product, an acknowledgment in the product documentation would be
  17. # appreciated but is not required.
  18. # 2. Altered source versions must be plainly marked as such, and must not be
  19. # misrepresented as being the original software.
  20. # 3. This notice may not be removed or altered from any source distribution.
  21. #
  22. # This script detects use of stdlib function in SDL code
  23. import argparse
  24. import os
  25. import pathlib
  26. import re
  27. import sys
  28. SDL_ROOT = pathlib.Path(__file__).resolve().parents[1]
  29. STDLIB_SYMBOLS = [
  30. 'abs',
  31. 'acos',
  32. 'acosf',
  33. 'asin',
  34. 'asinf',
  35. 'asprintf',
  36. 'atan',
  37. 'atan2',
  38. 'atan2f',
  39. 'atanf',
  40. 'atof',
  41. 'atoi',
  42. 'bsearch',
  43. 'calloc',
  44. 'ceil',
  45. 'ceilf',
  46. 'copysign',
  47. 'copysignf',
  48. 'cos',
  49. 'cosf',
  50. 'crc32',
  51. 'exp',
  52. 'expf',
  53. 'fabs',
  54. 'fabsf',
  55. 'floor',
  56. 'floorf',
  57. 'fmod',
  58. 'fmodf',
  59. 'free',
  60. 'getenv',
  61. 'isalnum',
  62. 'isalpha',
  63. 'isblank',
  64. 'iscntrl',
  65. 'isdigit',
  66. 'isgraph',
  67. 'islower',
  68. 'isprint',
  69. 'ispunct',
  70. 'isspace',
  71. 'isupper',
  72. 'isxdigit',
  73. 'itoa',
  74. 'lltoa',
  75. 'log10',
  76. 'log10f',
  77. 'logf',
  78. 'lround',
  79. 'lroundf',
  80. 'ltoa',
  81. 'malloc',
  82. 'memalign',
  83. 'memcmp',
  84. 'memcpy',
  85. 'memcpy4',
  86. 'memmove',
  87. 'memset',
  88. 'pow',
  89. 'powf',
  90. 'qsort',
  91. 'qsort_r',
  92. 'qsort_s',
  93. 'realloc',
  94. 'round',
  95. 'roundf',
  96. 'scalbn',
  97. 'scalbnf',
  98. 'setenv',
  99. 'sin',
  100. 'sinf',
  101. 'snprintf',
  102. 'sqrt',
  103. 'sqrtf',
  104. 'sscanf',
  105. 'strcasecmp',
  106. 'strchr',
  107. 'strcmp',
  108. 'strdup',
  109. 'strlcat',
  110. 'strlcpy',
  111. 'strlen',
  112. 'strlwr',
  113. 'strncasecmp',
  114. 'strncmp',
  115. 'strrchr',
  116. 'strrev',
  117. 'strstr',
  118. 'strtod',
  119. 'strtokr',
  120. 'strtol',
  121. 'strtoll',
  122. 'strtoul',
  123. 'strupr',
  124. 'tan',
  125. 'tanf',
  126. 'tolower',
  127. 'toupper',
  128. 'trunc',
  129. 'truncf',
  130. 'uitoa',
  131. 'ulltoa',
  132. 'ultoa',
  133. 'utf8strlcpy',
  134. 'utf8strlen',
  135. 'vasprintf',
  136. 'vsnprintf',
  137. 'vsscanf',
  138. 'wcscasecmp',
  139. 'wcscmp',
  140. 'wcsdup',
  141. 'wcslcat',
  142. 'wcslcpy',
  143. 'wcslen',
  144. 'wcsncasecmp',
  145. 'wcsncmp',
  146. 'wcsstr',
  147. ]
  148. RE_STDLIB_SYMBOL = re.compile(rf"\b(?P<symbol>{'|'.join(STDLIB_SYMBOLS)})\b\(")
  149. def find_symbols_in_file(file: pathlib.Path) -> int:
  150. match_count = 0
  151. allowed_extensions = [ ".c", ".cpp", ".m", ".h", ".hpp", ".cc" ]
  152. excluded_paths = [
  153. "src/stdlib",
  154. "src/libm",
  155. "src/hidapi",
  156. "src/video/khronos",
  157. "include/SDL3",
  158. "build-scripts/gen_audio_resampler_filter.c",
  159. "build-scripts/gen_audio_channel_conversion.c",
  160. "test/win32/sdlprocdump.c",
  161. ]
  162. filename = pathlib.Path(file)
  163. for ep in excluded_paths:
  164. if ep in filename.as_posix():
  165. # skip
  166. return 0
  167. if filename.suffix not in allowed_extensions:
  168. # skip
  169. return 0
  170. # print("Parse %s" % file)
  171. try:
  172. with file.open("r", encoding="UTF-8", newline="") as rfp:
  173. parsing_comment = False
  174. for line_i, original_line in enumerate(rfp, start=1):
  175. line = original_line.strip()
  176. line_comment = ""
  177. # Get the comment block /* ... */ across several lines
  178. while True:
  179. if parsing_comment:
  180. pos_end_comment = line.find("*/")
  181. if pos_end_comment >= 0:
  182. line = line[pos_end_comment+2:]
  183. parsing_comment = False
  184. else:
  185. break
  186. else:
  187. pos_start_comment = line.find("/*")
  188. if pos_start_comment >= 0:
  189. pos_end_comment = line.find("*/", pos_start_comment+2)
  190. if pos_end_comment >= 0:
  191. line_comment += line[pos_start_comment:pos_end_comment+2]
  192. line = line[:pos_start_comment] + line[pos_end_comment+2:]
  193. else:
  194. line_comment += line[pos_start_comment:]
  195. line = line[:pos_start_comment]
  196. parsing_comment = True
  197. break
  198. else:
  199. break
  200. if parsing_comment:
  201. continue
  202. pos_line_comment = line.find("//")
  203. if pos_line_comment >= 0:
  204. line_comment += line[pos_line_comment:]
  205. line = line[:pos_line_comment]
  206. if m := RE_STDLIB_SYMBOL.match(line):
  207. override_string = f"This should NOT be SDL_{m['symbol']}()"
  208. if override_string not in line_comment:
  209. print(f"{filename}:{line_i}")
  210. print(f" {line}")
  211. print(f"")
  212. match_count += 1
  213. except UnicodeDecodeError:
  214. print(f"{file} is not text, skipping", file=sys.stderr)
  215. return match_count
  216. def find_symbols_in_dir(path: pathlib.Path) -> int:
  217. match_count = 0
  218. for entry in path.glob("*"):
  219. if entry.is_dir():
  220. match_count += find_symbols_in_dir(entry)
  221. else:
  222. match_count += find_symbols_in_file(entry)
  223. return match_count
  224. def main():
  225. parser = argparse.ArgumentParser(fromfile_prefix_chars="@")
  226. parser.add_argument("path", default=SDL_ROOT, nargs="?", type=pathlib.Path, help="Path to look for stdlib symbols")
  227. args = parser.parse_args()
  228. print(f"Looking for stdlib usage in {args.path}...")
  229. match_count = find_symbols_in_dir(args.path)
  230. if match_count:
  231. print("If the stdlib usage is intentional, add a '// This should NOT be SDL_<symbol>()' line comment.")
  232. print("")
  233. print("NOT OK")
  234. else:
  235. print("OK")
  236. return 1 if match_count else 0
  237. if __name__ == "__main__":
  238. raise SystemExit(main())