Author Topic: some issues and requests  (Read 33362 times)

Offline oBFusCATed

  • Developer
  • Lives here!
  • *****
  • Posts: 13438
    • Travis build status
Re: some issues and requests
« Reply #45 on: October 23, 2010, 06:24:41 pm »
1. This is the right way.
2. We can not distinguish between  C or C++, but __cplusplus just needed by C++, not C. We can not find the best way.
3. This is impossible! Because in current, a project per to a parser.
4. Could you give me more explain?
5. In current, we record only the line number, but not the position.
6. Sorry, do not understood.
7. e.g. How to decision? void tes|t() { /* implement*/ }
1. No, it is not the right thing to do, I'm talking as a user.
   I don't care that some has used "typedef struct {} name;" (this is the correct C code, by the way), instead of "struct name {};".
   In the symbols browser I want to see the struct name, in both cases.
   Now, I get Unamed_Struct_00xxx and I should search in the typedefs to see what is the real name.
   You can say that it can't be done (of course I doubt about that:) ), but saying this is the right was is plain wrong :)
2. You can, it the .h file is included from C++ file the __cplusplus macro is define, simple as that (of course it will get pretty nasty if both C and C++ file include the same header) :)
3. I see
4. See here http://smrt.is-a-geek.org/codeblocks/screens/cc_toolbar_bug.png
5. Can you record the position, I don't think it will be such problem? (Another int in the token struct, big deal).
6. If you have code like this:

int func(int arg1)
{
 ....
 ....
    int c = arg1; <--- goto declaration here will goto the opening curly bracket of the function '{', not to the position of the argument
}
It looks like 6 is related to 5 :)

7. No need for decision, you can fill the popup menu with all the found declarations and implementations

Blueshake, seems you're right, probably the bug 9.1 is related to bug 10.
(most of the time I ignore long posts)
[strangers don't send me private messages, I'll ignore them; post a topic in the forum, but first read the rules!]

Offline ollydbg

  • Developer
  • Lives here!
  • *****
  • Posts: 5370
  • OpenCV and Robotics
    • Chinese OpenCV forum moderator
Re: some issues and requests
« Reply #46 on: October 24, 2010, 04:07:51 am »
5. Can you record the position, I don't think it will be such problem? (Another int in the token struct, big deal).
You mean the "column position"? if Yes, the Tokenizer class should record the column position, then add a int member variable in the Token class.


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.

Offline oBFusCATed

  • Developer
  • Lives here!
  • *****
  • Posts: 13438
    • Travis build status
Re: some issues and requests
« Reply #47 on: October 24, 2010, 12:29:58 pm »
Yes, the column
(most of the time I ignore long posts)
[strangers don't send me private messages, I'll ignore them; post a topic in the forum, but first read the rules!]

Offline Loaden

  • Lives here!
  • ****
  • Posts: 1014
Re: some issues and requests
« Reply #48 on: November 01, 2010, 09:53:29 am »
5. Goto declaration goes to the start of the line not the start of the token
6. Goto declaration goes to the { of a function, when the declaration is an argument, should go to the start of the token
8. Sometimes 'goto declaration' works but 'goto implementation' doesn't
13. Call tip sometimes don't show (pretty random)
14. There is no highlighted parameter in the calltip (sometimes)
Hi, oBFusCATed, these issue should be fixed in HEAD.

Offline Loaden

  • Lives here!
  • ****
  • Posts: 1014
Re: some issues and requests
« Reply #49 on: November 01, 2010, 09:57:16 am »
15. Symbols browser is not updated after some text is typed and "Update parser when typing" = on (when probably, should be replace with while (native speaker correct me please))
Can you post a test demo in here?
I can't reproduce you problem.

Offline Loaden

  • Lives here!
  • ****
  • Posts: 1014
Re: some issues and requests
« Reply #50 on: November 01, 2010, 10:06:11 am »
12. Call tip for constructors still not working
Fixed in r6802.

Offline Loaden

  • Lives here!
  • ****
  • Posts: 1014
Re: some issues and requests
« Reply #51 on: November 01, 2010, 10:27:05 am »
11. No template parameters for templated classes in the Symbol Browser
supported in r6803.

Offline Loaden

  • Lives here!
  • ****
  • Posts: 1014
Re: some issues and requests
« Reply #52 on: November 01, 2010, 10:33:36 am »
2. We can not distinguish between  C or C++, but __cplusplus just needed by C++, not C. We can not find the best way.
2. You can, it the .h file is included from C++ file the __cplusplus macro is define, simple as that (of course it will get pretty nasty if both C and C++ file include the same header) :)
No, we can't.
In batch parsing, we can only get the target compiler, but in a project, a target can per a compiler.
So, who is the C Compiler? who is the C++ compiler?
If there have C Compiler, and have C++ Compiler too, then how to decision?
This is too hard, we can't solved it.

Offline oBFusCATed

  • Developer
  • Lives here!
  • *****
  • Posts: 13438
    • Travis build status
Re: some issues and requests
« Reply #53 on: November 01, 2010, 12:34:01 pm »
Loaden: thanks for the fixes, I'll give it a try the next time Morten syncs the branches :)

No, we can't.
In batch parsing, we can only get the target compiler, but in a project, a target can per a compiler.
So, who is the C Compiler? who is the C++ compiler?
If there have C Compiler, and have C++ Compiler too, then how to decision?
This is too hard, we can't solved it.

Why do you care for the compiler?
You know the type of every translation unit (.c or .cpp file),
so if a header is included from a cpp file or a header where __cplusplus is define, you define the __cplusplus...
(most of the time I ignore long posts)
[strangers don't send me private messages, I'll ignore them; post a topic in the forum, but first read the rules!]

Offline Loaden

  • Lives here!
  • ****
  • Posts: 1014
Re: some issues and requests
« Reply #54 on: November 01, 2010, 01:24:53 pm »
Loaden: thanks for the fixes, I'll give it a try the next time Morten syncs the branches :)

No, we can't.
In batch parsing, we can only get the target compiler, but in a project, a target can per a compiler.
So, who is the C Compiler? who is the C++ compiler?
If there have C Compiler, and have C++ Compiler too, then how to decision?
This is too hard, we can't solved it.

Why do you care for the compiler?
You know the type of every translation unit (.c or .cpp file),
so if a header is included from a cpp file or a header where __cplusplus is define, you define the __cplusplus...
Another way, done in r6805.

Offline oBFusCATed

  • Developer
  • Lives here!
  • *****
  • Posts: 13438
    • Travis build status
Re: some issues and requests
« Reply #55 on: November 01, 2010, 01:41:34 pm »
Super :)
(most of the time I ignore long posts)
[strangers don't send me private messages, I'll ignore them; post a topic in the forum, but first read the rules!]

Offline oBFusCATed

  • Developer
  • Lives here!
  • *****
  • Posts: 13438
    • Travis build status
Re: some issues and requests
« Reply #56 on: December 29, 2010, 02:09:50 pm »
Another problem I've encountered:

Type "#include <wx/i", then the completion for the include shows, then choose wx/ipc.h from the list.
The result is "#include <wx/wx/ipc.h>" not the correct one "#include <wx/ipc.h>"
(most of the time I ignore long posts)
[strangers don't send me private messages, I'll ignore them; post a topic in the forum, but first read the rules!]

Offline jens

  • Administrator
  • Lives here!
  • *****
  • Posts: 7254
Re: some issues and requests
« Reply #57 on: December 29, 2010, 11:20:48 pm »
Another problem I've encountered:

Type "#include <wx/i", then the completion for the include shows, then choose wx/ipc.h from the list.
The result is "#include <wx/wx/ipc.h>" not the correct one "#include <wx/ipc.h>"

Works fine here on debian.

Offline oBFusCATed

  • Developer
  • Lives here!
  • *****
  • Posts: 13438
    • Travis build status
Re: some issues and requests
« Reply #58 on: December 29, 2010, 11:32:28 pm »
Hm, works here, too, now :(
It seems that it is a bit random or the testcase is time dependant :(
(most of the time I ignore long posts)
[strangers don't send me private messages, I'll ignore them; post a topic in the forum, but first read the rules!]