Fix reconstructed header dropping macros whose name/value contains the include guard (#1266)

[mkarlesky]

Fixes #1266. This is a port of #1271 (green on next_version) to master, for inclusion in 1.1.8.

Root cause

extract_macro_defs filtered a header's own include guard out of the reconstructed macro list with a plain substring test (macro.include?(include_guard)). That also rejected any ordinary macro whose name or value merely happened to contain the guard string as a substring — e.g. with guard RTC_H:

#define RTC_HOUR_SECONDS (60u * RTC_MINUTE_SECONDS)   // name contains "RTC_H"
#define RTC_DAY_SECONDS (24u * RTC_HOUR_SECONDS)       // value references RTC_HOUR_SECONDS, so it too contains "RTC_H"

Both were silently dropped from the reconstructed header, producing undeclared-identifier compile errors — exactly as reported.

Fix

The match is now anchored to #define <guard> followed by whitespace or end-of-line, so only the guard's own definition line matches.

Tests

Same coverage as #1271: new #extract_macro_defs cases in preprocessinator_reconstructor_spec.rb using the reporter's own RTC example, verified failing against pre-fix code, passing after. Unit tests only, no system tests. bundle exec rake specs:units clean (1991 examples, 0 failures, 1 pre-existing unrelated pending).

🤖 Generated with Claude Code