My favorites | Sign in
Project Home Downloads Wiki Issues Source
New issue   Search
for
  Advanced search   Search tips   Subscriptions
Issue 182: Insertion of unwanted tags
1 person starred this issue and may be notified of changes. Back to list
Status:  Accepted
Owner:  cds...@gmail.com


Sign in to add a comment
 
Reported by DHilgart...@gmail.com, Jan 27, 2012
Agent Smith inserts tags it shouldn't, sometimes only after multiple reformats.

Example:
Reformatting this:

/// <returns> 0 in case everything is ok, -1 otherwise. </returns>

yields this:

/// <returns>
/// <list type="number">
/// <item><description>in case everything is ok, -1 otherwise.</description></item>
/// </list>
/// </returns>

It seems to convert to leading 0 in the comment to a numbered list item...



Reformatting this:

    /// <summary>
    ///   The Fluent NHibernate auto-mapping override class for the <see cref="BookingCostCenter" /> entity.
    /// </summary>

yields this (note the new langword tag):

    /// <summary>
    /// The Fluent NHibernate auto-mapping <see langword="override"/> class for the <see cref="BookingCostCenter" />
    /// entity.
    /// </summary>

reformatting again yields this (note the extra space at the end of the previously inserted tag and the inserted para tags):

    /// <summary>
    /// <para>
    /// The Fluent NHibernate auto-mapping <see langword="override" /> class for the <see cref="BookingCostCenter" />
    /// </para>
    /// <para>entity.</para>
    /// </summary>

Settings:
Max characters per line: 120
Place tags on new line: "Long lines only" for all tags.


Jan 27, 2012
Project Member #1 cds...@gmail.com
Re the first - this was due to the list detection code getting a bit carried away. I'll include some changes in the next beta to only start lists if it sees "<number>. " or "<number> - " at the start of a line.

Re the second, adding the langword tag is intended behavior. You could add a regex in the options to ignore that identifier if you don't want that behavior.

Re the last, I'll look into it - probably something to do with ending the previous line with a tag.
Status: Accepted
Jan 27, 2012
Project Member #2 cds...@gmail.com
(No comment was entered for this change.)
Owner: cds...@gmail.com
Jan 28, 2012
#3 DHilgart...@gmail.com
Re the first: "<number>) " would also be a good candidate. Maybe make in configurable.
Re the second: It looks like it is not working correctly anyway. Because a lot of other similar words don't get replaced, like "this" or "object".
Re the third: Yes, I think the tag at the end of the line seems to be the problem. If there is no tag at the end of the line this doesn't happen.
Sign in to add a comment

Powered by Google Project Hosting