News:

The new Release 25.03 is out! You can download binaries for Windows and many major Linux distros here .

Main Menu

wxAuiManager preview issue under wxSmith cause C::B can't exit correctly

Started by ollydbg, February 11, 2019, 10:26:14 AM

Previous topic - Next topic

oBFusCATed

1. Do not use lambda captures if you don't understand them.
2. Why do you need a capture in this case? The frame is passed as a parameter.
3. What does Temp mean? To me it is the same as using My, Mine, Random, Some, etc.
(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!]

ollydbg

Quote from: oBFusCATed on February 13, 2019, 08:02:04 PM
1. Do not use lambda captures if you don't understand them.
Oh, I need to learn it. Should be easy.  :)
Quote
2. Why do you need a capture in this case? The frame is passed as a parameter.
You are correct, no need capture here.

Quote
3. What does Temp mean? To me it is the same as using My, Mine, Random, Some, etc.
What about shortLiveFrame or transientFrame?
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.

oBFusCATed

(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!]

ollydbg

Quote from: oBFusCATed on February 14, 2019, 07:06:57 PM
transientFrame has different meaning in some OS/wms.
OK, thanks.

I now use the "shortLiveFrame". I have committed the patch to trunk now(r11571). Thanks all guys for your help.
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.

BlueHazzard

Hi,
i can not build the latest C::B trunk on linux, because of the error message
function is not a member of std
To fix this i have to add
#include <functor>
at the top of the edited files

ollydbg

Quote from: BlueHazzard on February 17, 2019, 02:15:33 PM
Hi,
i can not build the latest C::B trunk on linux, because of the error message
function is not a member of std
To fix this i have to add
#include <functor>
at the top of the edited files
Sorry about that, because I don't use Linux most of the time. This does not happen on my Windows GCC 7.2. I think you can push your changes. Thanks.
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.

oBFusCATed

@bluehazzard: There is no <functor> header it is <functional> ...
(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!]

BlueHazzard

Quote from: oBFusCATed on February 17, 2019, 03:55:18 PM
@bluehazzard: There is no <functor> header it is <functional> ...
yes, my fault. It is <functional>

Quote from: ollydbg on February 17, 2019, 03:10:26 PM
Sorry about that, because I don't use Linux most of the time. This does not happen on my Windows GCC 7.2. I think you can push your changes. Thanks.
I think this is because windows uses precompiled headers? And linux not?
I will add the headers only in the cpp source files where it is needed

BlueHazzard

Well here we are.... I have commited with the wrong message text... Is there a way to edit the commit message?

ollydbg

Quote from: BlueHazzard on February 17, 2019, 04:14:46 PM
Well here we are.... I have commited with the wrong message text... Is there a way to edit the commit message?
Yes, you can edit the commit message. Under Windows, you can do this in TortoiseSVN, and right click of that commit, it will shown in context menu.

PS: I remember I did this several years ago. Now, I don't use TortoiseSVN for a long time. :)
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.

BlueHazzard


sodev

Quote from: BlueHazzard on February 17, 2019, 04:05:41 PM
I think this is because windows uses precompiled headers? And linux not?
More likely that a different code path pulls in different transitive includes, thats why you should always make your includes self contained :).