A patch for the php language support
Escuder Nicolas
n.escuder at intra-links.com
Wed Apr 13 09:18:05 UTC 2005
Thanks for your comment i made it.
See You
Nicolas.
-----Message d'origine-----
De : kdevelop-devel-admin at barney.cs.uni-potsdam.de
[mailto:kdevelop-devel-admin at barney.cs.uni-potsdam.de] De la part de Matt
Rogers
Envoyé : mardi 12 avril 2005 20:49
À : kdevelop-devel at kdevelop.org
Cc : Escuder Nicolas
Objet : Re: A patch for the php language support
On Tuesday 12 April 2005 03:05 am, Escuder Nicolas wrote:
> Hello,
>
>
>
> Here you can find a pacth that :
>
>
>
> - Add support for parsing include/require file from php script for seeing
> it in the class view browser.
>
> - Add an option in project options for the include_path for searching
> include files.
>
> - Correct an regexp for searching functions in classes.
>
> - Correct parsefile function because class view browser need absolute path
> not relative one.
>
>
>
> Sorry for my poor English i'm French.
>
>
>
> See You soon ;-)
>
>
>
> Nicolas Escuder
>
>
I don't use PHP, but I will comment on the the code.
In phpsupportpart.h, you make the PHPConfigData pointer public. I would
recommend that you don't do this, and instead, keep it protected and provide
an accessor function, and change all the places you reference the
PHPConfigData pointer to use that function.
Hope this helps,
Matt
More information about the KDevelop-devel
mailing list