2025-07-22 13:51 CEST

View Issue Details Jump to Notes ]
IDProjectCategoryView StatusLast Update
0002022Spring engineGeneralpublic2010-08-06 22:03
Reporterhoijui 
Assigned ToKloot 
PrioritynormalSeverityblockReproducibilityN/A
StatusresolvedResolutionfixed 
Product Version0.82.0+git 
Target VersionFixed in Version0.82.1 
Summary0002022: no const sync casts
Descriptionnot sure if i interpreted correctly, but that is what it looks like to me (see compile logs, further down).
during recent changes, some sync checks are now down on const vars, instead of mutable ones.
example: rts/Sim/MoveTypes/GroundMoveType.cpp:1952
relevant defines: rts/System/Sync/Upcast.h

if the problem is really that we have no templates defined for the const version of the types: is there an other way (eg, "ommitting" the const?), then to defined all the macros for const as well? it would result in 2 tiems the unary, and 4 times the binary templates, if i got that right.

if there is no other way, i will do that (am happy for that kind of boring and brainless work these days ;-) )
Additional Information
two compile logs, from different machines with different compiler versions:
http://pastebin.com/3qDbr8wi
http://pastebin.com/q1S2MG8E
TagsNo tags attached.
Checked infolog.txt for Errors
Attached Files

-Relationships
+Relationships

-Notes

~0005227

Kloot (developer)

Last edited: 2010-08-06 21:53

"if the problem is really that we have no templates defined for the const version of the types"

Yes and no. Those templates indeed don't exist, but things like

template<class U> SyncedPrimitive& operator>>=(const SyncedPrimitive& f) {x>>=f.x; Sync(">>="); return *this;}

would fail even if they did, so I'll just remove the const.

+Notes

-Issue History
Date Modified Username Field Change
2010-08-06 21:26 hoijui New Issue
2010-08-06 21:51 Kloot Note Added: 0005227
2010-08-06 21:53 Kloot Note Edited: 0005227
2010-08-06 22:03 Kloot Status new => resolved
2010-08-06 22:03 Kloot Fixed in Version => 0.82.1
2010-08-06 22:03 Kloot Resolution open => fixed
2010-08-06 22:03 Kloot Assigned To => Kloot
+Issue History