suricata icon indicating copy to clipboard operation
suricata copied to clipboard

Include less v11

Open catenacyber opened this issue 1 year ago • 1 comments

Link to redmine ticket: https://redmine.openinfosecfoundation.org/issues/945

Describe changes:

  • Remove unused includes in c files
  • adds some options in CI : hyperscan, luajit

Follows #7686 with smarter script paying attention to ifdefs and not changing anything when it gets too complex

The script may be run with other configure options to remove even more includes

There may be a preliminary commit to add that was squashed into main commits

catenacyber avatar Sep 16 '22 12:09 catenacyber

WARNING:

field baseline test %
build_asan

Pipeline 9228

suricata-qa avatar Sep 16 '22 12:09 suricata-qa

Requires #7911

catenacyber avatar Sep 23 '22 13:09 catenacyber

The include minimizer script should be more clever about placing includes in ifdefs : it should place them after the unconditional includes

catenacyber avatar Sep 23 '22 13:09 catenacyber

Replaced by https://github.com/OISF/suricata/pull/7933

catenacyber avatar Sep 26 '22 15:09 catenacyber