use const in parameters when applicable

macressler / bwapi

Automatically exported from code.google.com/p/bwapi

1 stars 0 forks source link

use const in parameters when applicable #453

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago

Normally I use const everywhere I can. This has, however, brought some minor 
problems when using BWAPI. For example Unit::isInWeaponRange(Unit*) const, does 
not use const on the parameter.

I guess this goes for some unit commands too, like Unit::attack(Unit*), but 
maybe not for Unit::load(Unit*).

I have solved these issues for now with some const_casts. :)

Original issue reported on code.google.com by senth.wa...@gmail.com on 15 Aug 2012 at 4:14

GoogleCodeExporter commented 9 years ago

Original comment by AHeinerm on 20 Aug 2012 at 2:16

Changed state: Started
Added labels: ****
Removed labels: ****

GoogleCodeExporter commented 9 years ago

Original comment by AHeinerm on 13 Dec 2012 at 8:38

Added labels: Priority-Low
Removed labels: Priority-Medium

GoogleCodeExporter commented 9 years ago

Original comment by AHeinerm on 16 Dec 2012 at 5:03

Changed state: Paused
Added labels: ****
Removed labels: ****

GoogleCodeExporter commented 9 years ago

The BWAPI project is slowly undergoing the migration to GitHub. All issues have 
been migrated to the GitHub issue tracker located here: 
https://github.com/bwapi/bwapi/issues

In order to receive updates on your issue, please head over to the GitHub issue 
tracker and subscribe to your issue.

Original comment by AHeinerm on 15 Apr 2014 at 7:46

Changed state: Migrated
Added labels: ****
Removed labels: ****