azbycxdw909 / garglk

Automatically exported from code.google.com/p/garglk
Other
0 stars 0 forks source link

Code not C99-compliant. Breaks compilation with clang #131

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
What steps will reproduce the problem?
1. Change compiler to clang
2. Compilation error

Clang compiles to the C99 standard by default, while gcc compiles to the C89 
standard.

Two interpreters break compilation: JACL and AdvSys.
* The JACL patch is here: https://code.google.com/p/jacl/issues/detail?id=9
* The AdvSys patch is attached.

Original issue reported on code.google.com by saltyho...@gmail.com on 9 Nov 2010 at 10:15

Attachments:

GoogleCodeExporter commented 8 years ago
Thanks for the patches!

Original comment by bcressey@gmail.com on 15 Nov 2010 at 5:57

GoogleCodeExporter commented 8 years ago
Fixed in r469.

Can you tell me your name, or how you would like to be credited?

Original comment by bcressey@gmail.com on 29 Nov 2010 at 7:57

GoogleCodeExporter commented 8 years ago
"Ori Avtalion" is fine, thanks :)

Original comment by saltyho...@gmail.com on 29 Nov 2010 at 8:04

GoogleCodeExporter commented 8 years ago
Added you to project credits, and updated JACL to 2.8.2 which includes your 
patch.

Original comment by bcressey@gmail.com on 29 Nov 2010 at 9:01