News:

As usual while waiting for the next release - don't forget to check the nightly builds in the forum.

Main Menu

Rev. 9438 question

Started by Alpha, November 08, 2013, 02:49:14 PM

Previous topic - Next topic

Alpha

By this new method, if we have __UnnamedStruct111 and file #1 this is the 11th unnamed struct, but file #11 has unnamed struct 1.  Are they merged?  Should we instead use unnamedTmp.Printf(_T("%s%s%lu_%lu"), ...) so we have __UnnamedStruct1_11 and __UnnamedStruct11_1 ?

oBFusCATed

I think we must find a way to remove these __UnnamedStruct entries at all...
(most of the time I ignore long posts)
[strangers don't send me private messages, I'll ignore them; post a topic in the forum, but first read the rules!]

ToApolytoXaos

Quote from: Alpha on November 08, 2013, 02:49:14 PM
By this new method, if we have __UnnamedStruct111 and file #1 this is the 11th unnamed struct, but file #11 has unnamed struct 1.  Are they merged?  Should we instead use unnamedTmp.Printf(_T("%s%s%lu_%lu"), ...) so we have __UnnamedStruct1_11 and __UnnamedStruct11_1 ?
Alpha, may I ask where did you find this? I'm interested in such peculiar things and I would like to take a look at it.

Alpha

Relevant changes are:
http://cb.biplab.in/websvn/comp.php?repname=codeblocks&compare[]=/@9437&compare[]=/@9438
(Found when reading through the log messages.)

ollydbg

Quote from: Alpha on November 08, 2013, 02:49:14 PM
By this new method, if we have __UnnamedStruct111 and file #1 this is the 11th unnamed struct, but file #11 has unnamed struct 1.  Are they merged?  Should we instead use unnamedTmp.Printf(_T("%s%s%lu_%lu"), ...) so we have __UnnamedStruct1_11 and __UnnamedStruct11_1 ?
Hi, Alpha, thanks for the review, indeed this is a bug. I fix this bug in rev 9443 by adding an underscore between file index and token index.
If some piece of memory should be reused, turn them to variables (or const variables).
If some piece of operations should be reused, turn them to functions.
If they happened together, then turn them to classes.