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