fix for python template

[Tommaso Massimi]

On 4/11/06, Matt Rogers <mattr at kde.org> wrote:
> On Monday 10 April 2006 07:23, tom wrote:
> > Hi all,
> >
> > I saw that the python application generated by the kdevelop wizard
> > is unable to use the dialog to open a file.
> >
> > Here a (simple) fix to the tempalte against the current svn revision.
> > Bye
> > Tom
> >
> > ===================================================================
> > --- languages/python/app_templates/pyqt/app.py  (revision 528198)
[...]

> Hi,
>
> I committed your patch.
>
> Thanks!

you are welcome,
it seems now that kdevelop has somebody keeping cure of it!!

Some time ago I sent also some few lines of code against a bug,

here the bug
http://bugs.kde.org/show_bug.cgi?id=113379

here the discussion in the mail list and the code I sent.

http://barney.cs.uni-potsdam.de/mailman/private/kdevelop-devel/2005-December/034339.html

I well know my fix was not the final word to fix that fault,
moreover it is affording the trouble in a way that solve the problem
in my context,
and I think it could be useful to everyone.

I'm keeping these lines on my kdevelop since a few months now,
so I'm sure they are working fine.

Can you let me have your opinion about them?

---

here the code, as I saw your webmail server is not helping so much...

 languages/cpp/problemreporter.cpp
 void ProblemReporter::closedFile(const KURL &fileName)
 {
-       QValueList<Problem> problems =
m_cppSupport->backgroundParser()->problems( fileName.path() , true ,
true);
+       KMimeType::Ptr res = KMimeType::findByFileContent( fileName.path() );
+       if (res && res->isValid())
+       {
+               QString mimetype = res->name();
+               if ((mimetype == "text/x-chdr"  ) ||
+                   (mimetype == "text/x-c++hdr") ||
+                   (mimetype == "text/x-csrc"  ) ||
+                   (mimetype == "text/x-c++src"))
+               {
+                       QValueList<Problem> problems =
m_cppSupport->backgroundParser()->problems( fileName.path() , true ,
true);
+               }
+       }
 }

Ciao
Tom