Code::Blocks Forums

Developer forums (C::B DEVELOPMENT STRICTLY!) => Development => CodeCompletion redesign => Topic started by: Loaden on February 12, 2011, 10:48:44 am

Title: Need more test for "one parser per whole workspace"
Post by: Loaden on February 12, 2011, 10:48:44 am: Hi, All!
During this time I really was busy, so this work has been delayed for too long.
Now I basically do it, but I need more tests.

Please post the feedback to here, thanks!
Title: Re: Need more test for "one parser per whole workspace"
Post by: ollydbg on February 15, 2011, 04:59:17 am: which means we only use "one tokenstree which holding all the projects in the workspace"?
I will test it!!!!
Title: Re: Need more test for "one parser per whole workspace"
Post by: MortenMacFly on February 15, 2011, 06:23:18 am: Quote from: Loaden on February 12, 2011, 10:48:44 am
Please post the feedback to here, thanks!
I am running C::B with this patch applied already, but I don't know what to do to enable it. I have a workspace with two application projects referencing 3 library projects. If I open the WS it seems only the active project is parsed, just as it was before. What do I have to do to test it?
Title: Re: Need more test for "one parser per whole workspace"
Post by: ollydbg on February 15, 2011, 06:24:52 am: Quote from: MortenMacFly on February 15, 2011, 06:23:18 am
Quote from: Loaden on February 12, 2011, 10:48:44 am
Please post the feedback to here, thanks!
I am running C::B with this patch applied already, but I don't know what to do to enable it. I have a workspace with two application projects referencing 3 library projects. If I open the WS it seems only the active project is parsed, just as it was before. What do I have to do to test it?
~~there is an option in CC's dialog to enable it. :D~~
sorry, I can't find this option either.
@loaden
How to enable it...
Title: Re: Need more test for "one parser per whole workspace"
Post by: Loaden on February 15, 2011, 07:40:26 am: Quote from: MortenMacFly on February 15, 2011, 06:23:18 am
Quote from: Loaden on February 12, 2011, 10:48:44 am
Please post the feedback to here, thanks!
I am running C::B with this patch applied already, but I don't know what to do to enable it. I have a workspace with two application projects referencing 3 library projects. If I open the WS it seems only the active project is parsed, just as it was before. What do I have to do to test it?
1. Settings > Editor > Code Completion > C/C++ Parser
2. Enable the new option named "... per whole workspace"

Then open cb's workspace, and active some project.
Title: Re: Need more test for "one parser per whole workspace"
Post by: ollydbg on February 15, 2011, 09:32:49 am: ok, find it, thanks!!
Title: Re: Need more test for "one parser per whole workspace"
Post by: Loaden on February 15, 2011, 12:28:35 pm: Update to V3, support parsing all project one by one when opened a workspace, e.g. CB's workspace.
Works well in here, but need more test.
Any feedback are welcome!
Title: Re: Need more test for "one parser per whole workspace"
Post by: MortenMacFly on February 15, 2011, 09:05:59 pm: Quote from: Loaden on February 15, 2011, 12:28:35 pm
Any feedback are welcome!
Generally it seems to work, hence for the UI part I would welcome the following:

- add a short explanation that by default it will be one parser per project (next to selecting the number of parsers)
- position the checkbox more to the left aligning it with label
- rename the label like "Use only one parser per workspace"
- when selecting the checkbox (or, if active) disable the choice for the number of parsers so it gets clear that either the one or the other is possible.
Title: Re: Need more test for "one parser per whole workspace"
Post by: Loaden on February 16, 2011, 10:46:49 am: Quote from: MortenMacFly on February 15, 2011, 09:05:59 pm
- add a short explanation that by default it will be one parser per project (next to selecting the number of parsers)
Oh, This is so hard to me. Because my poor english.
I can't explain well.:(
Title: Re: Need more test for "one parser per whole workspace"
Post by: MortenMacFly on February 16, 2011, 11:20:19 am: Quote from: Loaden on February 16, 2011, 10:46:49 am
I can't explain well.:(
No problem, we can add this later.
Title: Re: Need more test for "one parser per whole workspace"
Post by: Loaden on February 22, 2011, 06:39:12 am: Quote from: MortenMacFly on February 16, 2011, 11:20:19 am
Quote from: Loaden on February 16, 2011, 10:46:49 am
I can't explain well.:(
No problem, we can add this later.
Applied in rev7018, bacause I can't find any issue about this patch since V3.