View Issue Details [ Jump to Notes ] | [ Issue History ] [ Print ] | ||||||||
ID | Project | Category | View Status | Date Submitted | Last Update | ||||
---|---|---|---|---|---|---|---|---|---|
0002022 | Spring engine | General | public | 2010-08-06 21:26 | 2010-08-06 22:03 | ||||
Reporter | hoijui | ||||||||
Assigned To | Kloot | ||||||||
Priority | normal | Severity | block | Reproducibility | N/A | ||||
Status | resolved | Resolution | fixed | ||||||
Product Version | 0.82.0+git | ||||||||
Target Version | Fixed in Version | 0.82.1 | |||||||
Summary | 0002022: no const sync casts | ||||||||
Description | not 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 | ||||||||
Tags | No tags attached. | ||||||||
Checked infolog.txt for Errors | |||||||||
Attached Files |
|
![]() |
|
Kloot (developer) 2010-08-06 21:51 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. |
![]() |
|||
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 |