cleanup: remove unused offset field from OverloadFTOps struct#71
cleanup: remove unused offset field from OverloadFTOps struct#71Koan-Bot wants to merge 1 commit intocpan-authors:mainfrom
Conversation
The `int offset` member was never read or written anywhere in the XS or header code — not in any BOOT initializer, operator handler, or Perl-side function. Removing it eliminates dead memory (sizeof(int) per instance) and the confusion it causes for readers of the struct. Fixes cpan-authors#64 Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
|
@Koan-Bot review |
PR Review — cleanup: remove unused offset field from OverloadFTOps structThe SummaryThe Automated review by Kōan |
|
LGTM. I verified via grep that the |
2 participants
Summary
Removes the
int offsetmember from theOverloadFTOpsstruct inFileCheck.h. The field was never read or written anywhere in the XS or header code — not in anyBOOTinitializer, operator handler, or Perl-side function — making it dead code that wastes memory and misleads readers.Fixes #64
Changes
int offset;fromOverloadFTOpsstruct inFileCheck.hTest plan
make test— 1250 tests across 55 files, all greenGenerated by Kōan /fix
Quality Report
Changes: 1 file changed, 1 deletion(-)
Code scan: clean
Tests: passed (0 Tests)
Branch hygiene: 1 issue(s)
offsetfield from OverloadFTOps structGenerated by Kōan post-mission quality pipeline