I just discovered comparing values of type Val_Int32 and Val_UInt32 no longer makes sense given the lattice semantics. This means that we must modify our...
Hi Tyson, Can you be a bit more specific? Is it Val_Int32::compareTo(Val_UInt32) that doesn't work correctly (and what does it do?) Is it a type choice by the...
I'm personally confused as to how it's decided which operand of the comparator is "self". Regardless, there seems to be no enforcement in Pel that the...
I believe we've discussed this before. The compareTo method is there to induce a total order on ALL values. It is not intended to allow comparisons with...
Please see my previous email. compareTo is not meant to perform any casts whatsoever. And besides (but rather unrelated), it should absolutely not be the case...
My example was equality. I don't believe it is fully symmetric in the current implementation. My general point was that type promotion logic should be uniform...
The problem is with index lookups, which do not use the oper.* code and instead directly call the compareTo routines. Tyson ... The problem is with index...
To be more specific, the CommonTable::KeyedEntryComparator defined in p2core/commonTable.C directly calls the compareTo routines. An alternative would use the...
I don't know this code well enough to judge just now. But I think this whole discussion suggests that >1 person should sit down together and revisit this...
Done. I have assigned it to Petros to review. Thanks, Tyson ... Done. I have assigned it to Petros to review. Thanks, Tyson On May 9, 2007, at 9:19 PM, Joe...
Well, I consider this (using compareTo to sort table tuples) a feature, not a bug. I would not want my table's index to be performing implicit casts behind my...
We are talking about two different things: You are talking about numbers and I'm talking about classes. We made a design decision a while back that kept the...
Gents (and by that I mean Tyson and Eric), Is the TysonPetrosMerge branch supposed to be compilable? Or have you migrated your development to another branch? ...
OK, I've removed dlparser. However, the problem persists with runOverLog2 and runStagedOverLog as well. I'm not asking you to fix this, I can try doing that....
Petros, these are my settings: CXXFLAGS = -g -O2 LDFLAGS="" could you tell me what's the latest complaint msg? E. ... in? ... the ... Behalf ... files ... ...
Don't know if this helps debugging, but.... The move2cmake branch appears to build fine with static P2 libraries on grouchy. I'm not sure what else changed...
I'd guess that statically linking the non-p2 libraries is a lower priority. Putting in statically the P2 libraries removes the need for "installing" P2 on...
Folks: I have to miss tomorrow's P2 meeting I'm afraid. Also, I will be travelling the weeks of 6/4 and 6/11. In general, I'd like to move P2 meetings to Weds...
Having problems with message search?
Fill out this form to ensure your group is one of the first to be migrated to the new message search system.
