E53F Sync copyright notice and some minor cleanups by N-R-K · Pull Request #49 · mwh/dragon · GitHub
[go: up one dir, main page]
Skip to content

Sync copyright notice and some minor cleanups #49

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Sync copyright notice and some minor cleanups #49

wants to merge 5 commits into from

Conversation

N-R-K
Copy link
Contributor
@N-R-K N-R-K commented Apr 7, 2022

Hi,

This syncs the copyright notice on the comments with the output of --version.

Also some minor cleanups, details provided in the commit messages.

N-R-K added 5 commits April 7, 2022 20:12
@N-R-K
sync copyright notice with --version
a8efb20 
the copyright notice on the --version output was updated on 07fd608.
but the comment on the top of the file remained outdated.
@N-R-K
use proper conversion specifer for size_t
d21c84a
@N-R-K
avoid unnecessary heap allocation
3e5f234
@N-R-K
check for malloc failures
618ff8b
@N-R-K
avoid using function with unspecified arguments
6fb2870 
in C, fun(void) means this function takes _no arguments_, and it would
be a compile time error if this function was called with one.

however fun() means the function takes _unspecified_ arguments, so the
compiler can't do any type checking. calling such a function with
_any_ amount or any type of arguments will still end up compiling.

for example:

	void fun1()     { return; }
	void fun2(void) { return; }

	int
	main(void)
	{
		fun1(10, "string"); /* should compile fine */
		fun2(1); /* compile time error */
	}
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@N-R-K
0