News:

When registered with our forums, feel free to send a "here I am" post here to differ human beings from SPAM bots.

Main Menu

Need more test for "one parser per whole workspace"

Started by Loaden, February 12, 2011, 10:48:44 AM

Previous topic - Next topic
Print
Go Down Pages1
User actions

Loaden

February 12, 2011, 10:48:44 AM Last Edit: February 12, 2011, 05:57:40 PM by Loaden
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!

ollydbg

#1
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!!!!
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.

MortenMacFly

#2
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?
Compiler logging: Settings->Compiler & Debugger->tab "Other"->Compiler logging="Full command line"
C::B Manual: [url="https://www.codeblocks.org/docs/main_codeblocks_en.html"]https://www.codeblocks.org/docs/main_codeblocks_en.html[/url]
C::B FAQ: [url="https://wiki.codeblocks.org/index.php?title=FAQ"]https://wiki.codeblocks.org/index.php?title=FAQ[/url]

ollydbg

#3
February 15, 2011, 06:24:52 AM Last Edit: February 15, 2011, 06:40:18 AM by ollydbg
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...
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.

Loaden

#4
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.

ollydbg

#5
February 15, 2011, 09:32:49 AM
ok, find it, 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.

Loaden

#6
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!

MortenMacFly

#7
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.
Compiler logging: Settings->Compiler & Debugger->tab "Other"->Compiler logging="Full command line"
C::B Manual: [url="https://www.codeblocks.org/docs/main_codeblocks_en.html"]https://www.codeblocks.org/docs/main_codeblocks_en.html[/url]
C::B FAQ: [url="https://wiki.codeblocks.org/index.php?title=FAQ"]https://wiki.codeblocks.org/index.php?title=FAQ[/url]

Loaden

#8
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.:(

MortenMacFly

#9
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.
Compiler logging: Settings->Compiler & Debugger->tab "Other"->Compiler logging="Full command line"
C::B Manual: [url="https://www.codeblocks.org/docs/main_codeblocks_en.html"]https://www.codeblocks.org/docs/main_codeblocks_en.html[/url]
C::B FAQ: [url="https://wiki.codeblocks.org/index.php?title=FAQ"]https://wiki.codeblocks.org/index.php?title=FAQ[/url]

Loaden

#10
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.
Print
Go Up Pages1
User actions