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

Compilation warnings in Lua #4096

Closed
mtango688 opened this issue May 6, 2016 · 7 comments

Comments

Projects
None yet
4 participants
@mtango688
Copy link
Contributor

commented May 6, 2016

I recently tried the latest git on Ubuntu 16.04 amd64, and this warning came up:

/home/mtango/minetest/src/lua/src/loadlib.c: In function ‘ll_sym’: /home/mtango/minetest/src/lua/src/loadlib.c:76:21: warning: ISO C forbids conversion of object pointer to function pointer type [-Wpedantic] lua_CFunction f = (lua_CFunction)dlsym(lib, sym); ^

Note that this is using gcc 5.3.1, cmake 3.5.1.

@est31

This comment has been minimized.

Copy link
Contributor

commented May 6, 2016

Well its a lua issue, we can't do much about it, as we prefer not to edit around in different librarie's code... The only thing we should modify here is the cmake script.

Have you turned on some specific warnings? Or is that the default config now?

@mtango688

This comment has been minimized.

Copy link
Contributor Author

commented May 6, 2016

I don't know, but I guess pedantic is the new default.

@mtango688

This comment has been minimized.

Copy link
Contributor Author

commented May 6, 2016

There was some discussion upstream, now that I'm looking around for it.

http://lua-users.org/lists/lua-l/2015-01/msg00019.html

@Zeno-

This comment has been minimized.

Copy link
Contributor

commented May 6, 2016

That particular warning has actually been occurring for a long time (and always without specifically providing -Wpedantic). shrug

@sfan5 sfan5 changed the title Compile of loadlib.c gives warning ISO C forbids conversion of certain pointer types Compilation warnings in Lua May 7, 2016

@paramat paramat added the @ Build label May 7, 2016

@paramat

This comment has been minimized.

Copy link
Member

commented May 7, 2016

Yes i'm used to seeing that during build.

@mtango688

This comment has been minimized.

Copy link
Contributor Author

commented May 8, 2016

@paramat paramat closed this May 8, 2016

@Zeno-

This comment has been minimized.

Copy link
Contributor

commented May 8, 2016

Closed because it's a known "issue" and something that upstream should address, not us

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.