Attnam / ivan

Iter Vehemens ad Necem - a continuation of the graphical roguelike by members of http://attnam.com
GNU General Public License v2.0
301 stars 43 forks source link

Fix arguments in various ABORT messages #549

Closed ryfactor closed 5 years ago

attnam-bot commented 5 years ago

This pull request fixes 12 alerts when merging 7cf440a22c4aff4173a9ae101971a9599aaad923 into b12dd0cc3e7506e95ebceeda0eb2fa95392fe515 - view on LGTM.com

fixed alerts:


Comment posted by LGTM.com

attnam-bot commented 5 years ago

This pull request introduces 2 alerts and fixes 20 when merging 56e90f1b604b62e2ea7534f8b9b038470b769246 into b12dd0cc3e7506e95ebceeda0eb2fa95392fe515 - view on LGTM.com

new alerts:

fixed alerts:


Comment posted by LGTM.com

attnam-bot commented 5 years ago

This pull request fixes 18 alerts when merging 08f06626b3470cd52fd2887fde7ee47a3aa3c9cc into b12dd0cc3e7506e95ebceeda0eb2fa95392fe515 - view on LGTM.com

fixed alerts:


Comment posted by LGTM.com

attnam-bot commented 5 years ago

This pull request fixes 18 alerts when merging 0a70c76126c1d7cbb384aa07e3365361f1992c57 into b12dd0cc3e7506e95ebceeda0eb2fa95392fe515 - view on LGTM.com

fixed alerts:


Comment posted by LGTM.com

jakwings commented 5 years ago

What are LGTM's compiler flags?

attnam-bot commented 5 years ago

This pull request fixes 20 alerts when merging 0d0316879b3ed15038e84f5a999cf3abffbc3234 into b12dd0cc3e7506e95ebceeda0eb2fa95392fe515 - view on LGTM.com

fixed alerts:


Comment posted by LGTM.com

ryfactor commented 5 years ago

What are LGTM's compiler flags?

I'm not particularly sure. There's such thing as an lgtm project configuration file that offers some control over configurations, but I don't know exactly what level of stringency their warnings go to.

jakwings commented 5 years ago

I forgot to squash the commits. :p