Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 93403

Summary:	[Wizards] Support pluggable comment providers
Product:	[Eclipse Project] Platform	Reporter:	Luigi Viggiano <lviggiano>
Component:	CVS	Assignee:	platform-cvs-inbox <platform-cvs-inbox>
Status:	RESOLVED WONTFIX	QA Contact:
Severity:	enhancement
Priority:	P4	CC:	otaviofp
Version:	3.1	Keywords:	helpwanted
Target Milestone:	---
Hardware:	PC
OS:	Windows XP
Whiteboard:

Description Luigi Viggiano

2005-05-02 09:34:07 EDT

Hi.

Is it possible to have a feature like this:

http://www.jroller.com/resources/jigi/subclipse-default-comment.png

where 
+ means add
- means removed
~ means changed 

also default message could be helpful, like, for example
"fixed imports" (imports are modified)

etc.

This could be very handly and speeding-up synchronizing.

Luigi

Comment 1 Michael Valenta

2005-05-02 11:06:50 EDT

This is one of many interesting ways to format your comments. However, I don't 
think it is generally useful (i.e. I would never use it since I can always wee 
what changed by comparing). What would be more interesting is to allow third 
parties to plug in there own comment generator/formatter. I don't have the 
time to look at this myself but I'll leave it open in cae someone else is 
interested in implementing it.

Comment 2 Luigi Viggiano

2005-05-20 23:07:53 EDT

yes: you can get difference doing diffs manually. but you can also do a cvs log
and get a structure behaviour. When you get prompted with

+createWallet(Wallet)
+loadWallet(Integer)
~save(Wallet)

you can add details. before committing ie:

+createWallet(Wallet):optimized implementation
+loadWallet(Integer):throw exception if it's not found
~save(Wallet): added check on unique code before update

having a "basic" comment could help, anyway.

Pluggable default comments would be great. But... someone has to implement an AI
algorithm that interprets changes and generate comments :-). 
You know I'm really lazy.

Thanks
Luigi.

Comment 3 Luigi Viggiano

2005-05-20 23:12:19 EDT

Also, if you are checking in from other views (for example, from package or
navigator), if the checkbox is selected you get the:

+createWallet(Wallet)
+loadWallet(Integer)
~save(Wallet)

that is confortable as you don't have to diff to get this summary (that,
obviously  should be integrated with more meaningful informations)

Also, better some poorly speaking comments than any.

Comment 4 Michael Valenta

2006-06-19 15:54:08 EDT

There is currently no plan to address this item.

Comment 5 Denis Roy

2009-08-30 02:16:10 EDT

As of now 'LATER' and 'REMIND' resolutions are no longer supported.
Please reopen this bug if it is still valid for you.