Skip to content

Remove compat snprintf() (eggdrop requires posix 2001) - #1942

Open
michaelortmann wants to merge 4 commits into
eggheads:developfrom
michaelortmann:snprintf
Open

michaelortmann wants to merge 4 commits into
eggheads:developfrom
michaelortmann:snprintf

Conversation

@michaelortmann

@michaelortmann michaelortmann commented Oct 6, 2026 •

Copy link
Copy Markdown
Member

Found by: vanosg
Patch by: michaelortmann
Fixes:

One-line summary:
Eggdrop depends on POSIX.1-2001, which defines vsnprintf().

Additional description (if needed):
This PR keeps egg_snprintf() for backward compatibility to (older) modules but redefines it to use snprintf() directly with a one-liner define in eggdrop.h.

All egg_snprintf() were replaced by snprintf():
find ./ -type f -exec sed -i 's/egg_snprintf(/snprintf(/g' {} \;
Note: don't do this in git directory. cd src/ first!

This PR will lead to better compiler diagnostics, new warnings during make, and codacy static code analysis findings, as expected. Follow-UP PRs should fix them to keep this very PR small and thus easier to review.

Test cases demonstrating functionality (if applicable):
No functional change. I tested 2 share bots, which successfully started and shared userfile.

@michaelortmann

Copy link
Copy Markdown
Member Author

This PR leads to better diagnostics / more compiler warnings. Now i fixed some of those, the easy to fix ones. The remaining ones are also easy to fix but will need more changes / code lines, so i leave those to a Follow-up PR to help the review-process.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant