-
Notifications
You must be signed in to change notification settings - Fork 15
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
Code should be updated to ANSI C #3
Comments
The link-time errors about missized objects can be solved by adding the |
In my fork (http://github.com/pdewacht/macross) I have sort of a rough start to this. I used GCC's protoize tool to add function prototypes, fixed the thus-discovered errors (though not the warnings), and converted the vararg-like functions to actual varargs. I'm not submitting this as a pull request, as the work is pretty rough, and I don't think I'm going to spend further time on this. Still, it's a useful start for somebody who's willing to put serious effort in this. |
I'm looking through the results, and it's very promising so far. The vast majority of the remaining warnings are int-conversion and incompatible-pointer-type warnings. It's good that those have stuck around - I think those actually will explicitly mark the places where the code is not 64-bit clean. |
I've started exploiting the results from gcc aux-info (which protoize uses under the hood) and this is coming along pretty nicely. It's developing at https://github.com/michaelcmartin/macross/tree/ansi-c so far, and I've got Macross largely in shape and Slinky should follow shortly. |
This has several subtasks:
The text was updated successfully, but these errors were encountered: