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: show comments in VA Outline
 New Topic  Reply to Topic
 Printer Friendly
Author Previous Topic Topic Next Topic  

Uniwares
Tomato Guru

Portugal
2322 Posts

Posted - Jul 15 2008 :  11:04:30 AM  Show Profile  Reply with Quote
"Show Comments" in the VA Outline has some problems with .NET style comments.

this
/// <summary>
/// A function
/// </summary>
/// <param name="p1">some parameter</param>
/// <param name="p2">another one</param>
/// <returns>void</returns>


Turns into:
// <summary> / A function / </summary> / <param name="p1">some parameter</param> / <param name="p2">another one</param> / <returns>void</returns>

feline
Whole Tomato Software

United Kingdom
19020 Posts

Posted - Jul 15 2008 :  1:46:31 PM  Show Profile  Reply with Quote
I agree this is not very attractive, but short of parsing the comment I am not sure what we could do.

The problem with parsing the comment is what do we show? Also it opens the door to people asking VA to parse any structured comment, which once you look at C++ and Doxygen is not something we really want to get into.

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

Uniwares
Tomato Guru

Portugal
2322 Posts

Posted - Jul 15 2008 :  3:30:21 PM  Show Profile  Reply with Quote
True, but where do the single slashes come from? I guess VA simply removes // (double slashes) from the comments before concatenating. So it could maybe watch out for triple slashes and xml comments and strip them away?
Go to Top of Page

feline
Whole Tomato Software

United Kingdom
19020 Posts

Posted - Jul 16 2008 :  1:21:11 PM  Show Profile  Reply with Quote
A very good point, I have put in a bug report for the single slashes being left over:

case=18539

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

support
Whole Tomato Software

5566 Posts

Posted - Dec 01 2008 :  07:46:57 AM  Show Profile  Reply with Quote
case=18539 is fixed in build 1707
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