VcsJob question and suggestion

[Matthew Woehlke]

Andreas Pakulat wrote:
> On 29.05.07 13:49:36, Matthew Woehlke wrote:
>> dukju ahn wrote:
>>> What if  other plain actions such as commit() add(), copy(), ??
>>> Is it worth for them to implement fetchResults() ?? I don't think so.
>>> Only retrieving FinishStatus would be sufficient for them.
>> We could have it return null, or perhaps have VcsJob and VcsJobEx (bad 
>> names, can't think of anything better ATM), where the latter subclasses 
>> the former (but is still an interface) and adds fetchResults. Opinions? 
>> Other suggestions?
> 
> All wrong :) Thats where QVariant::invalid() is really helpful, just
> return QVariant() when the job doesn't have a result value.

s/null/"something that means 'no value'"/ in my original message, which 
is what I meant :-). IOW I think we agree that returning "something that 
means 'no value'" is adequate.

-- 
Matthew
"A mouse is a device used to point at the xterm you want to type in."
   --Kim Alm, A.S.R.