Whole Tomato Software Forums
Whole Tomato Software Forums
Main Site | Profile | Register | Active Topics | Members | Search | FAQ
User name:
Password:
Save Password
Forgot your password?

 All Forums
 Visual Assist
 Technical Support
 1645: bad coloring in forward declarations
 New Topic  Reply to Topic
 Printer Friendly
Author Previous Topic Topic Next Topic  

.oisyn
Tomato Guru

162 Posts

Posted - Jul 14 2008 :  6:18:27 PM  Show Profile  Reply with Quote
When you forward-declare a class or struct, that type is colored like a variable instead of a type.

struct Foo;
struct Bar
{
};

Foo is colored as a variable, while Bar is colored like a type.

(related to http://forum.wholetomato.com/forum/topic.asp?TOPIC_ID=7982 ?)

Edited by - .oisyn on Jul 14 2008 6:20:31 PM

feline
Whole Tomato Software

United Kingdom
19020 Posts

Posted - Jul 15 2008 :  09:39:54 AM  Show Profile  Reply with Quote
Can you try this test code please:

struct FelineForwardDeclareChocolateStruct;
struct FelineCreateChocolateStruct
{
};


both types are coloured as types for me. I have picked these names since they should be unique. If the forward declared type name is also the name of a variable VA is probably getting confused over exactly what it is, and how to colour it.

zen is the art of being at one with the two'ness
Go to Top of Page

.oisyn
Tomato Guru

162 Posts

Posted - Jul 15 2008 :  5:55:51 PM  Show Profile  Reply with Quote
You are correct. But there are no variables with those names in that scope - even if I use a completely empty sourcefile with just 'struct A;' it colors A as a variable. This is new in 1645 and pretty annoying. If you define a struct rather than declaring it, coloring is fine too. Why is a declaration different from a definition? Both things declare a type which from thereon can be used, so the distinction makes no sense in the first place.

Edited by - .oisyn on Jul 15 2008 6:00:41 PM
Go to Top of Page

feline
Whole Tomato Software

United Kingdom
19020 Posts

Posted - Jul 16 2008 :  10:15:09 AM  Show Profile  Reply with Quote
Which version of VA were you using before? Using the following C++ code and VS2005 I have just run some tests.

struct A;
struct forwardDeclareVariableName;

static void testGeneralCode()
{
	int forwardDeclareVariableName;
}


"A" is coloured as a variable in VA 1626, 1640 and 1645.
"forwardDeclareVariableName" is coloured as a type in 1626 on both lines - which is wrong.
"forwardDeclareVariableName" is coloured as a variable in 1640 and 1645 on both lines, which is still wrong.

However given the limitations our colouring code has to work with this is an unfair test.

In your code is the forward declared type ever actually defined anywhere? There is a difference between a forward declaration and an actual declaration. Specifically you do not want alt-g to take you to the forward declaration of a type.

zen is the art of being at one with the two'ness
Go to Top of Page

.oisyn
Tomato Guru

162 Posts

Posted - Jul 24 2008 :  5:13:18 PM  Show Profile  Reply with Quote
Hmmm, I could swear I have never seen it happen before 1645. But indeed, as soon as the type is actually defined somewhere (even in a completely unrelated file), it gets colored as a type. Btw, the actual code I was seeing it in was a case where a (private) nested struct was declared in the header and defined in the sourcefile. However, this no longer seems to be the case, and there might have been a complete reparse of the symbols in between, so perhaps an out-dated database was to blame for that.

But, as we are on the subject anyway, doing an alt-g on a type in a header that was only declared in that header brings me to the declaration in that header. While in the sourcefile, where it's defined, it brings me to the definition of the type in the sourcefile.

Example:
// foo.h

struct Foo;  // #1
void Bar(Foo f);  // #2


// foo.cpp
#include "foo.h"

struct Foo { };  // #3
void Bar(Foo f)  // #4
{
}

If I do alt-g on 'Foo' in #2, it brings me to #1. If I do it on 'Foo' in #4, it brings me to #3. Of course I'd rather have it to navigate to #3 if I'm on #2 (or even #1) as well.

Edited by - .oisyn on Jul 24 2008 5:15:35 PM
Go to Top of Page

feline
Whole Tomato Software

United Kingdom
19020 Posts

Posted - Jul 25 2008 :  1:04:47 PM  Show Profile  Reply with Quote
Does Find References help in this situation?

zen is the art of being at one with the two'ness
Go to Top of Page

.oisyn
Tomato Guru

162 Posts

Posted - Jul 27 2008 :  1:15:52 PM  Show Profile  Reply with Quote
No. If I do a find references on the 'Foo' at #1 or #2 it finds nothing. If I do it on the 'Foo' at #3 or #4 it finds all uses of Foo (even those in the header).

Btw, I just found out that coloring doesn't work for template classes of which only specializations are defined as well, but only if those classes have constructors:
template<class T> struct Foo;
template<class T> struct Bar;


template<> struct Foo<int>
{
};

template<> struct Bar<int>
{
    Bar() { }
};

Foo will be colored as a type, but Bar isn't (as a method actually), until I define a general template of Bar (with or without ctor).
Go to Top of Page

feline
Whole Tomato Software

United Kingdom
19020 Posts

Posted - Aug 01 2008 :  4:09:15 PM  Show Profile  Reply with Quote
Find References on the forward declared structure, I am seeing the same problem:

case=18883

And I am also seeing the colouring problem with the specialised template:

case=18884

zen is the art of being at one with the two'ness
Go to Top of Page

support
Whole Tomato Software

5566 Posts

Posted - Jan 31 2010 :  1:38:50 PM  Show Profile  Reply with Quote
case=18883 is fixed in build 1810

Whole Tomato Software, Inc.
Go to Top of Page
  Previous Topic Topic Next Topic  
 New Topic  Reply to Topic
 Printer Friendly
Jump To:
© 2023 Whole Tomato Software, LLC Go To Top Of Page
Snitz Forums 2000