http://bugzilla.novell.com/show_bug.cgi?id=480862

User [email protected] added comment
http://bugzilla.novell.com/show_bug.cgi?id=480862#c5


Eberhard Beilharz <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[email protected]




--- Comment #5 from Eberhard Beilharz <[email protected]>  2009-05-19 13:57:09 MDT 
---
(In reply to comment #4)

I agree that it's a workaround, not a real fix. Doing real syntax checking
would be better.

However, it matches the way the 'working' variant is treated. I can write:

/// <seealso cref="4::::B;;;B```````````434343"/>

or

/// <seealso cref="!:List&lt;*&gt;"/>

and don't get any warning or error.

Since the current behavior of not accepting Bla{T} or Bla&lt;T&gt; breaks
compatibility with csc I'd really like to see this patch applied to the code -
maybe add a FIXME comment to check the syntax later.

-- 
Configure bugmail: http://bugzilla.novell.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
_______________________________________________
mono-bugs maillist  -  [email protected]
http://lists.ximian.com/mailman/listinfo/mono-bugs

Reply via email to