<div class="gmail_quote">2012/3/14 Milian Wolff <span dir="ltr"><<a href="mailto:mail@milianw.de" target="_blank">mail@milianw.de</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div>On Wednesday 14 March 2012 10:00:07 Miha Čančula wrote:<br>
> Ok guys, I believe I have something ready to be merged. Currently, this<br>
> means:<br>
> * kdevplatform:<br>
>   - three new interfaces: ITest{Controller, Provider, Suite}<br>
>   - an ITestController implementation as a plugin<br>
>   - another plugin for the test view<br>
>   - a "run tests" button in Patch Review<br>
> * kdevelop:<br>
>   - CTest provider/runner plugin<br>
>   - Changes to the CMake parsing to store information about test added with<br>
> add_test()<br>
<br>
</div>the above sound fine<br>
<div><br>
> * kdev-php<br>
>   - PHPUnit provider/runner plugin<br>
>   - Again, changes to the parsing, so that unit the PHPUnit class<br>
> definitions are imported into every file.<br>
<br>
</div>This sounds wrong. Why do you need to import that into every file? I'd rather<br>
check the dependency chain on a QualifiedIdentifier basis for the PHPUnit base<br>
class. But I'll have to see whether that works out.<br></blockquote><div>Yeah, I agree. I didn't know a better way to do it, and I'm still not sure what the right way is. Now I removed that and instead disabled the "only files in opened projects" check in helper.cpp. This way, declarations from the 'phpunitdeclarations.php' file can be found and imported as a parent context. <br>
<br>But it still doesn't work perfectly, it only finds some of the cases. I don't know why. <br> </div><blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">

<div><br>
> Some of them are only additions (all the plugins and the new interfaces),<br>
> but there are some modifications to the existing code. I would like to get<br>
> some feedback on those, especially the language parsing, because I'm still<br>
> quite new to that. Would you prefer to look at the code in the branch,<br>
> three large ReviewBoard requests, or should I make separate review requests<br>
> for every component?<br>
<br>
</div>I'll review it soon I hope, the branches are fine for reviewing such large<br>
change sets I think.<br>
<br>
Thanks!<br>
<div><div><br>
--<br>
Milian Wolff<br>
<a href="mailto:mail@milianw.de" target="_blank">mail@milianw.de</a><br>
<a href="http://milianw.de" target="_blank">http://milianw.de</a></div></div><br>--<br>
KDevelop-devel mailing list<br>
<a href="mailto:KDevelop-devel@kdevelop.org" target="_blank">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>
<br></blockquote></div><br>