[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kdevelop-devel
Subject:    [VCS] removing showXXX
From:       Andreas Pakulat <apaku () gmx ! de>
Date:       2007-11-13 17:33:38
Message-ID: 20071113173338.GA5634 () morpheus ! apaku ! dnsalias ! org
[Download RAW message or body]

Hi,

I was about to implement showDiff() for Subversion and already found it
cool to replace the current slot for the diff-context menu with just a
call to showDiff. But then I found some "questions":

a) how to show the widget
b) how's the lifespan of the job and the widget object
c) how to let the widget know about the job

Especially c) leads one directly to something like:

DiffWidget( VcsJob* job, QWidget* parent );

And when you're there, you don't actually need showDiff anymore, because
it doesn't provide anything you can't do with the
kdevplatform-diffwidget and the normal diff() call. (I know the
diffwidget doesn't exist yet).

Same problems arise for the other showXXX methods, namely log, annotate,
change, ls and cat. For all of those we will have a widget that can
display the result of the according VcsJob and thus the "extra work" to
display the result instead of doing something with it is practically
non-existent.

Also it removes some confusion from the API.

So are there any objections against removing the showXXX methods, do I
overlook anything?

Andreas

-- 
Future looks spotty.  You will spill soup in late evening.

_______________________________________________
KDevelop-devel mailing list
KDevelop-devel@kdevelop.org
https://barney.cs.uni-potsdam.de/mailman/listinfo/kdevelop-devel
[prev in list] [next in list] [prev in thread] [next in thread]