Enhance Interface Pointers

zloop1982 / bwapi

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

0 stars 0 forks source link

Enhance Interface Pointers #487

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago

Hiding the interface pointers behind an interface wrapper is actually a very 
important change which will make BWAPI appear more professional.

Similar to the current Game wrapper that overloads operator <<, the base 
wrapper will overload operator -> and store the pointer to the interface object.

Original issue reported on code.google.com by AHeinerm on 11 May 2013 at 6:30

GoogleCodeExporter commented 9 years ago

Original comment by AHeinerm on 15 May 2013 at 6:10

Added labels: Priority-Medium, Milestone-MajorRelease
Removed labels: Priority-Critical, Milestone-Release

GoogleCodeExporter commented 9 years ago

A simple typedef would probably be the easiest way to hide the pointers for now 
while maintaining the same functionality.

It will be the simplest to implement right now.

It really doesn't matter if it still resolves to a pointer, just getting rid of 
the * is good enough, and if necessary we can enhance it later by replacing the 
typedef with a formal class.

For bot developers, you simply need to "Find and Replace In Files"

"Unit*" -> "Unit"
"Unit *" -> "Unit"

Will do the trick.

Game/Broodwar already has its own interface wrapper for the operator << 
overload.

Original comment by AHeinerm on 20 May 2013 at 4:47

GoogleCodeExporter commented 9 years ago

Original comment by AHeinerm on 24 May 2013 at 3:06

Changed state: Started
Added labels: Maintainability

GoogleCodeExporter commented 9 years ago

Make interface pointers non-typedef.

Original comment by AHeinerm on 13 Aug 2013 at 1:23

Changed title: Enhance Interface Pointers
Added labels: Priority-Low
Removed labels: Priority-Medium, Milestone-MajorRelease

GoogleCodeExporter commented 9 years ago

Original comment by AHeinerm on 13 Aug 2013 at 1:23

Changed state: Paused

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