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

Non-prototype declarations #4

Closed
frozencemetery opened this issue Sep 7, 2011 · 3 comments
Closed

Non-prototype declarations #4

frozencemetery opened this issue Sep 7, 2011 · 3 comments

Comments

@frozencemetery
Copy link
Member

Issue migrated from trac ticket # 3

component: component1 | priority: major | resolution: fixed

2011-09-07 13:52:39: @greghudson created the issue


libverto's headers contain a number of function declarations which aren't prototypes because they use () instead of (void). In definitions this is okay, but in prototypes this causes the compiler to treat the declaration as a K&R-style declaration and not a prototype.

verto-glib.h:verto_ctx *verto_new_glib();
verto-glib.h:verto_ctx *verto_default_glib();
verto-libevent.h:verto_ctx *verto_new_libevent();
verto-libevent.h:verto_ctx *verto_default_libevent();
verto-libev.h:verto_ctx *verto_new_libev();
verto-libev.h:verto_ctx *verto_default_libev();
verto-module.h:typedef verto_ctx *(*verto_ctx_constructor)();
verto-tevent.h:verto_ctx *verto_new_tevent();
verto-tevent.h:verto_ctx *verto_default_tevent();
@frozencemetery
Copy link
Member Author

2011-09-26 10:10:10: @npmccallum changed status from new to closed

@frozencemetery
Copy link
Member Author

2011-09-26 10:10:10: @npmccallum changed resolution from ** to fixed

@frozencemetery
Copy link
Member Author

2011-09-26 10:10:10: @npmccallum commented


Fixed in f6dda6f.

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

No branches or pull requests

1 participant