Bug 93403 - [Wizards] Support pluggable comment providers
Summary: [Wizards] Support pluggable comment providers
Status: RESOLVED WONTFIX
Alias: None
Product: Platform
Classification: Eclipse Project
Component: CVS (show other bugs)
Version: 3.1   Edit
Hardware: PC Windows XP
: P4 enhancement (vote)
Target Milestone: ---   Edit
Assignee: platform-cvs-inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords: helpwanted
Depends on:
Blocks:
 
Reported: 2005-05-02 09:34 EDT by Luigi Viggiano CLA
Modified: 2009-08-30 02:16 EDT (History)
1 user (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Luigi Viggiano CLA 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 CLA 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 CLA 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 CLA 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 CLA 2006-06-19 15:54:08 EDT
There is currently no plan to address this item.
Comment 5 Denis Roy CLA 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.