Compile with ANSI options and fix compiler warnings

jhallen / cpm

Run CP/M commands in Linux/Cygwin with this Z80 / BDOS / ADM-3A emulator.

Other

108 stars 26 forks source link

Compile with ANSI options and fix compiler warnings #8

Closed JL2210 closed 5 years ago

JL2210 commented 5 years ago

Don't use C++ comments Fix unsigned/signed integer comparisons Enable all and extra warnings void out all unused arguments remove unnecessary arguments from functions where they won't be needed other small fixes

Let me know if there's anything else I can change.

JL2210 commented 5 years ago

@jhallen?

jhallen commented 5 years ago

Sorry, I merged then unmerged by accident. BTW, I noticed it was not working in Cygwin, so I made another change.

JL2210 commented 5 years ago

@jhallen What's the error? I think you merged the master branch into this branch instead of the other way around.