[prev in list] [next in list] [prev in thread] [next in thread]
List: kdevelop-devel
Subject: Re: fix for python template
From: Matt Rogers <mattr () kde ! org>
Date: 2006-04-30 3:10:38
Message-ID: 200604292210.38543.mattr () kde ! org
[Download RAW message or body]
On Tuesday 11 April 2006 07:10, Tommaso Massimi wrote:
> On 4/11/06, Matt Rogers <mattr@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-Decembe
>r/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
IIRC, this has already been taken care of in the KDevelop 3.4 branch. I'll
confirm and then close the bug.
--
Matt
_______________________________________________
KDevelop-devel mailing list
KDevelop-devel@barney.cs.uni-potsdam.de
http://barney.cs.uni-potsdam.de/mailman/listinfo/kdevelop-devel
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic