Developer forums (C::B DEVELOPMENT STRICTLY!) > CodeCompletion redesign

Patch: function arguments added to autocomplete tooltip

<< < (7/24) > >>

oBFusCATed:
p2rkw:
First impressions - it look really bad, when longer functions are completed. It adds '...' at the end.
Is it possible to try to resize the completion window?

p2rkw:
My first idea was to call:
            ed->GetControl()->AutoCompSetMaxWidth(0);
            ed->GetControl()->AutoCompSetMaxHeight(12);
before AutoComp show, but this doesn't work. Maximum size of autocomp listbox is hardcoded in ListBoxImpl::GetDesiredRect() in file PlatWX.cpp:
(...) if (maxw > 350)
        maxw = 350;
add similar for maxh.
This limitations are unnecessary because later scintilla clamp rect returned by this function to values setted by AutoCompSetMaxWidth and AutoCompSetMaxHeight. So I removed this limitations and AutoCompSetMaxWidth and AutoCompSetMaxHeight works as it should be.

p2rkw:
I removed few bugs, now default arguments, and overloads are handled correctly. If you use autocomp outside function's block argument's types will be not removed (because outside blocks are only function's definitions/declarations).
I also apped patch for PlatWX.cpp - if you apply it your autocomplete listbox will be adjusted to widest match.
I think I finished this functionality, now I will work on documentation helper.

edit: patches are now in patch tracker.

ollydbg:

--- Quote from: p2rkw on November 30, 2012, 03:00:49 pm ---patches are now in patch tracker.

--- End quote ---
Good work.
Both Enlargment of autocomplete's listbox and CC,autocomplete: added argument's names and types to list.
The first one was patch on PlatWX.cpp.

MortenMacFly:

--- Quote from: ollydbg on November 30, 2012, 03:43:11 pm ---
--- Quote from: p2rkw on November 30, 2012, 03:00:49 pm ---patches are now in patch tracker.

--- End quote ---
Good work.
Both Enlargment of autocomplete's listbox and CC,autocomplete: added argument's names and types to list.
The first one was patch on PlatWX.cpp.

--- End quote ---
Warning: These conflict with:
http://forums.codeblocks.org/index.php/topic,17172.msg117678.html#new
...which is a nice one, too.

Navigation

[0] Message Index

[#] Next page

[*] Previous page

Go to full version