void IBasicVersionControl::showXXXX()

[Andreas Pakulat]

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).

Andreas

-- 
Break into jail and claim police brutality.