Developer forums (C::B DEVELOPMENT STRICTLY!) > Development

"Goto Function" Window

(1/2) > >>

Bat:
I've provided 2 separate patchs for :

1)Try to enhance this window. Presentation is a mater of taste, comment ...
https://sourceforge.net/p/codeblocks/tickets/66/
Before

After


2)As I must always resize this window, another patch to save/restore windows position
https://sourceforge.net/p/codeblocks/tickets/65/
One (major) drawback, is that all theses positions will end up in .conf file
Second is that some race case (monitor layout change) have to be handled ...

oBFusCATed:
For 66:
1. You've left some commented code
2. The indentations look odd
3. There are no comments what you're doing
4. You're setting a font explicitly by name
5. You're using c style struct definitions, which look odd in c++
6. The sort function look slow
8. Why don't you use a wxlistctrl instead of wxlistbox?
9. Naming variables with two letters is rather bad
10. Prefer std::vector to wxarrays
11. The { is placed at the wrong place according to our minimal style guide

If you fix 8 probably most of the others won't be valid any more.
Also I'm not sure if your version of the UI is better so it is probably better idea to introduce an option.

For 65:
This patch looks rather dangerous to me. Who is responsible to create or destroy WindowClosed objects?
What is the value of the this pointer when the on close event is triggered?
What happens if the window is last created on the second monitor which is non-existent anymore!?

Bat:
Thanks for feedback

Ticket 66:
wxlistctrl which option seem the best ?
-Make a specific dialog on src/gotofunctiondlg.cpp (and add here option for enabling/disabling this behaviour)
-Add this 'list ctrl' "option" to IncrementalSelectListDlg (sdk/incrementalselectlistdlg.cpp)
-other ?

[Edited : wrong ticket number]

oBFusCATed:

--- Quote from: Bat on October 14, 2014, 08:47:03 am ----Make a specific dialog on src/gotofunctiondlg.cpp (and add here option for enablinvg/disabling this behaviour)

--- End quote ---
This would be the easier option.


--- Quote from: Bat on October 14, 2014, 08:47:03 am ----Add this 'list ctrl' "option" to IncrementalSelectListDlg (sdk/incrementalselectlistdlg.cpp)

--- End quote ---
This would probably require making the api hard to use or you'll have to add serious hacks to keep it compatible.

Bat:
Here is a fully reworked version of this patch :
https://sourceforge.net/p/codeblocks/tickets/66/

It's now a custom dialog gotofunctiondlg.cpp that handle standard goto function mode and "ColumnMode"
I've tried to correct all other formatting things.

Navigation

[0] Message Index

[#] Next page