Danek Duvall wrote:
Context diffs didn't display very well in the forum preview, or I would have included them.It seems like a worthwhile, if likely short-lived, RFE, but your patch seems not to be against the current version of wx, and the lack of any context makes it hard to figure out how it *would* apply.
Short lived? Is wx going away? Does Mecurial offer some other alternative.
I'm guessing that the argument to wx_show_comment is whether or not thecomment comments should be shown, right?
Yes.
Yes, the same function is used to create the putback comments and delget comments where the extra comments are currently not welcome.Is that really necessary, or is the function used for both in cases where you'd want to keep the commentsand in cases where they need to disappear?
If so, I'd choose a different method -- some sort of flag in the typical "-X" form.
:-) --no_comments
Danek
Cheers, Stace --Stacey Marshall. Solaris Revenue Product Engineering, EMEA -----------------------------------------------------------------------
http://blogs.sun.com/ace
smime.p7s
Description: S/MIME Cryptographic Signature
_______________________________________________ tools-discuss mailing list tools-discuss@opensolaris.org