Removing now redundant comment. The fix was correct.
svn-id: r46868
This commit is contained in:
parent
a22fd021e9
commit
be1d586cd5
1 changed files with 0 additions and 9 deletions
|
@ -333,15 +333,6 @@ public:
|
||||||
}
|
}
|
||||||
void setNoPendingVerb() {
|
void setNoPendingVerb() {
|
||||||
_pendingVerb = getVerbType(kVerbNone);
|
_pendingVerb = getVerbType(kVerbNone);
|
||||||
// TODO: Someone with knowledge of SAGA should review this.
|
|
||||||
// This initially looked like:
|
|
||||||
// _currentObject[0] = _currentObject[0] = ID_NOTHING;
|
|
||||||
// and thus was an undefined operation on _currentObject[0]
|
|
||||||
// according to the C(++) standard.
|
|
||||||
// Now the question is whether this should really reset
|
|
||||||
// both objets or just _currentObject[0].
|
|
||||||
// In case the current code is ok, just remove the TODO,
|
|
||||||
// otherwise fix the code and remove the TODO.
|
|
||||||
_currentObject[0] = _currentObject[1] = ID_NOTHING;
|
_currentObject[0] = _currentObject[1] = ID_NOTHING;
|
||||||
setPointerVerb();
|
setPointerVerb();
|
||||||
}
|
}
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue