Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fixed the function name of fgetws #103

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

abdoei
Copy link

@abdoei abdoei commented Oct 5, 2024

Thanks for taking the time to contribute to GCC! Please be advised that if you are
viewing this on github.com, the mirror there is unofficial and unmonitored.

I know that, it is a small fix.

This part of the code exposes the C functions of wchar. It seems that the writer of the edited line forgot to change the last character after duplicating the line and the compiler doesn't complain about duplicate using declaration (or it does but it definitely compiles without errors).

@abdoei
Copy link
Author

abdoei commented Oct 5, 2024

Sorry for not noticing that it is just a duplicate. :)

@Peter0x44
Copy link
Contributor

@abdoei GCC developers don't read patches here, you should submit it on the gcc-patches mailing list.
Doing so requires writing a ChangeLog for the commit message.
https://gcc.gnu.org/codingconventions.html#ChangeLogs
write that, then use git format-patch to generate a patch file, and attach that to your message.
See also:
https://gcc.gnu.org/contribute.html

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.

2 participants