Re: [PHP-DEV] Documentation (Doxygen)

2017-05-30 Thread Fleshgrinder
On 5/30/2017 9:26 PM, Stanislav Malyshev wrote: > Hi! > > Sorry if it sounded that way, I of course meant nothing like it. I just > meant that introducing docs standard should not be made in a routine > unrelated patch, where it could be missed by many people, but as an > ordered process.

Re: [PHP-DEV] Documentation (Doxygen)

2017-05-30 Thread Stanislav Malyshev
Hi! >> But randomly introducing docs system without any explicit decision in an >> unrelated patch doesn't look like a good idea to me. >> > > Wow! This sounds like you think that I am trying to deliberately > sabotaging the PHP project. Quite the opposite is the case. I am simply Sorry if it

Re: [PHP-DEV] Documentation (Doxygen)

2017-05-30 Thread Fleshgrinder
Hey Stas! On 5/30/2017 12:50 AM, Stanislav Malyshev wrote: > Hi! > >> I used Doxygen in both PRs to document the code. Right now the code base >> is lacking a lot of documentation, which, if done right, would greatly >> improve accessibility of the code base. > > Well, the problem as I

Re: [PHP-DEV] Documentation (Doxygen)

2017-05-29 Thread Stanislav Malyshev
Hi! > I used Doxygen in both PRs to document the code. Right now the code base > is lacking a lot of documentation, which, if done right, would greatly > improve accessibility of the code base. Well, the problem as I understand it is that we don't have Doxygen setup for docs generation. So,

Re: [PHP-DEV] Documentation (Doxygen)

2017-05-29 Thread Fleshgrinder
On 5/24/2017 12:33 PM, Fleshgrinder wrote: > Hey internals! > > Nikic recommended that we discuss this topic before my latest PRs can be > merged. > > https://github.com/php/php-src/pull/2523 > https://github.com/php/php-src/pull/2535 > > I used Doxygen in both PRs to document the code. Right

[PHP-DEV] Documentation (Doxygen)

2017-05-24 Thread Fleshgrinder
Hey internals! Nikic recommended that we discuss this topic before my latest PRs can be merged. https://github.com/php/php-src/pull/2523 https://github.com/php/php-src/pull/2535 I used Doxygen in both PRs to document the code. Right now the code base is lacking a lot of documentation, which, if