Class BuiltinMemcpyChecker

ci/lint_cpp/builtin_memcpy_checker.py:22–87

Checker for raw __builtin_memcpy — use FL_BUILTIN_MEMCPY instead.

class BuiltinMemcpyChecker(FileContentChecker):
    """Checker for raw __builtin_memcpy — use FL_BUILTIN_MEMCPY instead."""

    def __init__(self):
        self.violations: dict[str, list[tuple[int, str]]] = {}

    def should_process_file(self, file_path: str) -> bool:
        """Check if file should be processed."""
        if not file_path.endswith((".cpp", ".h", ".hpp", ".ino", ".cpp.hpp")):
            return False

        # Skip the file that defines FL_BUILTIN_MEMCPY
        if file_path.endswith("compiler_control.h"):
            return False

        # Skip third-party code
        if "/third_party/" in file_path.replace("\\", "/"):
            return False

        return True

    def check_file_content(self, file_content: FileContent) -> list[str]:
        """Check file content for raw __builtin_memcpy usage."""
        violations: list[tuple[int, str]] = []
        in_multiline_comment = False

        for line_number, line in enumerate(file_content.lines, 1):
            stripped = line.strip()

            # Track multi-line comment state
            if "/*" in line:
                in_multiline_comment = True
            if "*/" in line:
                in_multiline_comment = False
                continue

            if in_multiline_comment:
                continue

            if stripped.startswith("//"):
                continue

            code_part = line.split("//")[0]

            # Skip #define lines that define the macro wrapper itself
            if "#define" in code_part and "FL_BUILTIN_MEMCPY" in code_part:
                continue

            # Fast check before regex
            if "__builtin_memcpy" not in code_part:
                continue

            if BUILTIN_MEMCPY_PATTERN.search(code_part):
                violations.append(
                    (
                        line_number,
                        f"Use FL_BUILTIN_MEMCPY instead of __builtin_memcpy: {stripped}\n"
                        f"      Rationale: FL_BUILTIN_MEMCPY wraps __builtin_memcpy for portability.\n"