<div class="gmail_quote">On Thu, Apr 22, 2010 at 6:39 PM, Andreas Pakulat <span dir="ltr"><<a href="mailto:apaku@gmx.de">apaku@gmx.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<div><div></div><div class="h5">On 22.04.10 17:54:22, Aleix Pol wrote:<br>
> Hi,<br>
> There's a bug that's being triggered from the git plugin when commiting<br>
> sometimes.<br>
> The problem is in this line:<br>
> kdevplatform/vcs/widgets/vcscommitdialog.cpp:261. The git support is<br>
> returning a null job, I was about to add an if there just to check if it has<br>
> to be run but I was wondering then if we will have to do that on every case<br>
> and add documentation for that. So what do you think we should do there?<br>
><br>
> SVN and CVS suppoort never return null jobs, do they?<br>
<br>
</div></div>Without looking at the code: A vcs plugin should never return a null job,<br>
even if the parameters are wrong it should still create a job and let that<br>
job fail early on inside of start(). Hence git returning a null job for an<br>
interface method means its breaking the interface. This may mean we need to<br>
split out the function from the interface.<br>
<br>
Andreas<br>
<br>
--<br>
Your reasoning powers are good, and you are a fairly good planner.<br>
<font color="#888888"><br>
--<br>
KDevelop-devel mailing list<br>
<a href="mailto:KDevelop-devel@kdevelop.org">KDevelop-devel@kdevelop.org</a><br>
<a href="https://barney.cs.uni-potsdam.de/mailman/listinfo/kdevelop-devel" target="_blank">https://barney.cs.uni-potsdam.de/mailman/listinfo/kdevelop-devel</a><br>
</font></blockquote></div><br>good, that's the answer I wanted to get :P<br><br>I'll look into removing all this return 0 in git support :)<br>