void IBasicVersionControl::showXXXX()
Matthew Woehlke
mw_triad at users.sourceforge.net
Tue Jun 5 19:03:26 UTC 2007
Andreas Pakulat wrote:
> On 05.06.07 12:38:50, Matthew Woehlke wrote:
>> dukju ahn wrote:
>>> I think majority of us agreed that showXXXX() should be void, since
>>> it is totally async. When using this method, the caller doesn't need
>>> to take care of the operation -- call and forget operation.
>>>
>>> Is there any objection that I change this into void ??
>> I don't recall that consensus :-). My argument for returning a job is
>> you can wait on the user to finish using the GUI, e.g. if a script needs
>> to invoke a GUI method. But maybe others feel differently...
>
> Well, maybe this should be done on a case-by-case basis, for example
> I don't see a reason to wait for showDiff(), but for example
> showCommit() might make sense to wait for. (think of doing something
> after the commit is done).
Yes, I can agree with that. Also is there a detach() method of VcsJob
(i.e. something that says 'run, and I don't care about the result, clean
yourself up')?
--
Matthew
ESIG: .sig file not available
More information about the KDevelop-devel
mailing list