Closed ivanperez-keera closed 9 months ago
@swt2c The tests seem to pass: https://app.travis-ci.com/github/Copilot-Language/copilot/jobs/615203542#L1691-L1769
Can you please try this solution and let me know if it works?
I can confirm that fef0297 does indeed fix the test compile warnings related to integer literals, thanks!
Wonderful! Thank you, @swt2c!
Change Manager: Changes requested:
show
instance, but this is a CShow
instance.Implementor: Changes implemented. Review requested.
Change Manager: Verified that:
Solution is implemented:
[X] The code proposed compiles and passes all tests. Details: Build log: https://app.travis-ci.com/github/Copilot-Language/copilot/builds/267932537
[X] The solution proposed produces the expected result. Details: The issue cannot be easily reproduced because it is dependent on the architecture and compiler. The correct behavior has been verified by asking the user who reported the issue to confirm that they no longer experience the problem with the solution proposed. See https://github.com/Copilot-Language/copilot/pull/479#issuecomment-1862077333 for details.
This commit adjusts the tests for the
copilot-c99
library to use a portable way of printing constants in the C code generated by the tests, as prescribed in the solution proposed for #471.