check_stdlib_usage.py 6.6 KB

123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262
  1. #!/usr/bin/env python3
  2. #
  3. # Simple DirectMedia Layer
  4. # Copyright (C) 1997-2025 Sam Lantinga <slouken@libsdl.org>
  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. "src/video/stb_image.h",
  158. "include/SDL3",
  159. "build-scripts/gen_audio_resampler_filter.c",
  160. "build-scripts/gen_audio_channel_conversion.c",
  161. "test/win32/sdlprocdump.c",
  162. ]
  163. filename = pathlib.Path(file)
  164. for ep in excluded_paths:
  165. if ep in filename.as_posix():
  166. # skip
  167. return 0
  168. if filename.suffix not in allowed_extensions:
  169. # skip
  170. return 0
  171. # print("Parse %s" % file)
  172. try:
  173. with file.open("r", encoding="UTF-8", newline="") as rfp:
  174. parsing_comment = False
  175. for line_i, original_line in enumerate(rfp, start=1):
  176. line = original_line.strip()
  177. line_comment = ""
  178. # Get the comment block /* ... */ across several lines
  179. while True:
  180. if parsing_comment:
  181. pos_end_comment = line.find("*/")
  182. if pos_end_comment >= 0:
  183. line = line[pos_end_comment+2:]
  184. parsing_comment = False
  185. else:
  186. break
  187. else:
  188. pos_start_comment = line.find("/*")
  189. if pos_start_comment >= 0:
  190. pos_end_comment = line.find("*/", pos_start_comment+2)
  191. if pos_end_comment >= 0:
  192. line_comment += line[pos_start_comment:pos_end_comment+2]
  193. line = line[:pos_start_comment] + line[pos_end_comment+2:]
  194. else:
  195. line_comment += line[pos_start_comment:]
  196. line = line[:pos_start_comment]
  197. parsing_comment = True
  198. break
  199. else:
  200. break
  201. if parsing_comment:
  202. continue
  203. pos_line_comment = line.find("//")
  204. if pos_line_comment >= 0:
  205. line_comment += line[pos_line_comment:]
  206. line = line[:pos_line_comment]
  207. if m := RE_STDLIB_SYMBOL.match(line):
  208. override_string = f"This should NOT be SDL_{m['symbol']}()"
  209. if override_string not in line_comment:
  210. print(f"{filename}:{line_i}")
  211. print(f" {line}")
  212. print(f"")
  213. match_count += 1
  214. except UnicodeDecodeError:
  215. print(f"{file} is not text, skipping", file=sys.stderr)
  216. return match_count
  217. def find_symbols_in_dir(path: pathlib.Path) -> int:
  218. match_count = 0
  219. for entry in path.glob("*"):
  220. if entry.is_dir():
  221. match_count += find_symbols_in_dir(entry)
  222. else:
  223. match_count += find_symbols_in_file(entry)
  224. return match_count
  225. def main():
  226. parser = argparse.ArgumentParser(fromfile_prefix_chars="@")
  227. parser.add_argument("path", default=SDL_ROOT, nargs="?", type=pathlib.Path, help="Path to look for stdlib symbols")
  228. args = parser.parse_args()
  229. print(f"Looking for stdlib usage in {args.path}...")
  230. match_count = find_symbols_in_dir(args.path)
  231. if match_count:
  232. print("If the stdlib usage is intentional, add a '// This should NOT be SDL_<symbol>()' line comment.")
  233. print("")
  234. print("NOT OK")
  235. else:
  236. print("OK")
  237. return 1 if match_count else 0
  238. if __name__ == "__main__":
  239. raise SystemExit(main())