Request: wxAUI code adjustments

[libfab]

Hi,
I've been using the wxAUI branch with great satisfaction on linux / Kubuntu 09.04, for reasons that are clearly explained in this thread by Jens:
http://forums.codeblocks.org/index.php/topic,10552.msg72360.html

yet on this brach I'm confronted with an annoying and documented copy-and-paste bug, which since about a year has been removed from trunk thanks to sound thinking by killerbot, see thread here:
http://forums.codeblocks.org/index.php/topic,8890.0.html

and the follow-on here:
http://forums.codeblocks.org/index.php/topic,8984.0.html

My request is: could the wxAUI be permanently patched along the lines of the above thread, so that users like me can get rid of the (quite annoying) copy-and-paste bug?
Thanks in advance,
Fab

[Jenna]

I just tested it on my system (lxde+openbox, debian 64-bit bleeding edge, unstable + many parts of experimental) and have no problems.
I use drag and drop a lot, but copy/cut and paste also works without any problems.

I will test it on KDE 4.3 (debian unstabvle 64-bit) and probably on Kubuntu 8.04.

Can you specify more exactly what problems you have ?
Probably the setup of your sstem might be of interest (keyboard-layout, kde shortcuts etc., wxWidgets and C::B-version, from svn or nightlies).

EDIT:
No problems on kde 4.3 .
I used my nightlies and forced dependencies to work with wxWidgets from debian unstable (2.8.7).

[MortenMacFly]

I just tested it on my system (lxde+openbox, debian 64-bit bleeding edge, unstable + many parts of experimental) and have no problems.

Current scintilla branch should really work. I am aware that I removed the patches you described but I also remove the fiddeleing of wxScintilla with DnD. Basically DnD works now natively through scintilla and this should be reliable. So there is no need for the patch anymore unless you tell me otherwise.
Please try the scintilla branch on your system and give feedback... As Jens said: Most important is your system configuration.

BTW: The wxAUI branch is way old meanwhile and has been merged into trunk for some time now. You might want to update anyways...

[Jenna]

I just tested it on my system (lxde+openbox, debian 64-bit bleeding edge, unstable + many parts of experimental) and have no problems.

Current scintilla branch should really work. I am aware that I removed the patches you described but I also remove the fiddeleing of wxScintilla with DnD. Basically DnD works now natively through scintilla and this should be reliable. So there is no need for the patch anymore unless you tell me otherwise.
Please try the scintilla branch on your system and give feedback... As Jens said: Most important is your system configuration.

BTW: The wxAUI branch is way old meanwhile and has been merged into trunk for some time now. You might want to update anyways...

I should say: I tested it with the current trunk from svn (r5790).

[libfab]

Can you specify more exactly what problems you have ?
Probably the setup of your sstem might be of interest (keyboard-layout, kde shortcuts etc., wxWidgets and C::B-version, from svn or nightlies).

OK Jens, so I do confirm the issue with following config:

system:               Kubuntu 09.04 dist-upgraded, built with g++ 4.4.1
CB version:          svn build 5680 from wxAUI branch
keyboard-layout: fr, model pc105, no variant (this because of the independently documented 'French keyboard space bar bug'
kde shotcuts:      standard Kubuntu 09.04, none user defined
CB shortcuts:      I erased the Ctrl+X, Ctrl+V, Ctrl+C shortcuts in CB settings dialog, following your above thread advice. Unlike what happened in your thread report, this did not solve the issue.
$ wx-config --libs
-pthread -Wl,-Bsymbolic-functions  -lwx_gtk2u_richtext-2.8 -lwx_gtk2u_aui-2.8 -lwx_gtk2u_xrc-2.8 -lwx_gtk2u_qa-2.8 -lwx_gtk2u_html-2.8 -lwx_gtk2u_adv-2.8 -lwx_gtk2u_core-2.8 -lwx_baseu_xml-2.8 -lwx_baseu_net-2.8 -lwx_baseu-2.8
$ wx-config --version
2.8.9
wx packages from Ubuntu i386 debian repos.
config.log attached.

I've not tested trunk for a while, since I followed Jens' advice tu use wxAUI, owing to (way) shorter opening times for complex projects (see first thread above), a Linux platform issue.

Jens improved the opening times issue with a patch first applied to wxAUI (although the improvement is only partial as windows versions are still much faster).
Has it been ported to trunk ever since, so that I could now move back to trunk?
Thanks, Fab


[attachment deleted by admin]

[MortenMacFly]

Has it been ported to trunk ever since, so that I could now move back to trunk?

BTW: The wxAUI branch is way old meanwhile and has been merged into trunk for some time now. You might want to update anyways...

Please try and report back. Thanks.

[libfab]

Please try and report back. Thanks.

Report is: on parallel build of trunk (svn 5800) with unchanged wx config and platform:

  a)  trunk  does not have the copy-and-paste bug, unlike the wxAUI branch
  b)  trunk does open more slowly than wxAUI, which means Jens' clever patch was not applied to trunk, or not fully so.

Fab

[Jenna]

 b)  trunk does open more slowly than wxAUI, which means Jens' clever patch was not applied to trunk, or not fully so.

It is implemented.

What does more slowly mean ? 48 instead of 44 seconds, or is it really much slower ?

The project I tested for the patch (dvda-author) still opens in less than 50 seconds (about 45~48 s).
I had to change one small thing, to make the editor respect colour-settings for the whole background again, but I dir not get a measurable difference with or without the change.

I just tested it with actual trunk (svn r5813) compiled with gcc 4.4.1 (Debian 4.4.1-4) on 64-bit unstable, linked against wxWidgets 2.8.10 from apt.wxwidgets.org .

[libfab]

Right Jens, I measured again more carefully and yes, opening time is barely higher by a handful of seconds this time.
So I'm now moving to trunk, to avoid cut-and-paste issues on wxAUI.
Thanks for the tests, Fab