Skip to content

Removing pedantic flag - #28

Open
JiaT75 wants to merge 3 commits into
keithn:masterfrom
JiaT75:removing_pedantic_flag
Open

Removing pedantic flag#28
JiaT75 wants to merge 3 commits into
keithn:masterfrom
JiaT75:removing_pedantic_flag

Conversation

@JiaT75

@JiaT75 JiaT75 commented Nov 18, 2021

Copy link
Copy Markdown
Contributor

The pedantic flag is not compatible with the FUNCTION macro extension. This fix will remove the warnings generated when compiling.

@lewisporter

Copy link
Copy Markdown

Danger. The author of this PR is a known malicious actor.

@ZLima12

ZLima12 commented Mar 31, 2024

Copy link
Copy Markdown

The user has been observed weakening warnings, presumably so that any exploits they create go unnoticed. I wouldn't accept this.

@xealits

xealits commented Apr 6, 2024

Copy link
Copy Markdown

not only the warnings, the commit 7978e54 "Added configure to .gitignore since it should be regenerated" adds configure and autom4te.cache/ in .gitignore. Autoconf and m4 are arcane things. Is it really sound to move configure outside version control?

@johnzhou721 johnzhou721 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reject if I were you.

Comment thread .gitignore
bin/
config.log
config.status
configure

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
configure

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's malicious, you're going to take over make a release with changed configure

Comment thread .gitignore
config.log
config.status
configure
autom4te.cache/ No newline at end of file

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
autom4te.cache/
autom4te.cache/

# I am not sure what effect that has really. - guidod
for ac_arg dnl
in "-pedantic % -Wall -ansi -pedantic" dnl GCC
in "-ansi % -Wall -ansi" dnl GCC

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is wrong. Reject this

@nionis

nionis commented Feb 27, 2026

Copy link
Copy Markdown
image

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.

6 participants