I can't test it actually, but il seems fine for me.
Best,
Yannick
2013/9/18 Chris Wright
> On Tue, Sep 17, 2013 at 5:08 PM, Yannick Torrès
> wrote:
>
> > I just push a patch for this :
>
> Great, thanks :-)
>
> On a related note, can you review
> https://github.com/php/web-doc-editor/pull/2 ?
On Tue, Sep 17, 2013 at 5:08 PM, Yannick Torrès
wrote:
> I just push a patch for this :
Great, thanks :-)
On a related note, can you review
https://github.com/php/web-doc-editor/pull/2 ?
This has come about because I just reverted an invalid edit on
en/make_chm_index.html and it is now showing
I just push a patch for this :
http://git.php.net/?p=web/doc-editor.git;a=commitdiff;h=46ed4c5754dc2ed8a541c3168d9b97c7db08c871
It will solve '...' argument warning, but not the "en/language/oop5/decon.xml"
warning ;)
2013/9/17 Chris Wright
> On Tue, Sep 17, 2013 at 12:34 PM, Yannick Torrès
On Tue, Sep 17, 2013 at 12:34 PM, Yannick Torrès
wrote:
> Could you please give me a filename as example for this ?
en/reference/array/functions/array-intersect-ukey.xml
en/reference/array/functions/array-multisort.xml
Interestingly, the following file also has one but doesn't complain about it
About decon.xml, the tag is necessary for the check. See the regex
from the last modified file from my last commit
Best,
Yannick
2013/9/17 Yannick Torrès
> I just push a patch for this :
>
>
> http://git.php.net/?p=web/doc-editor.git;a=commitdiff;h=46ed4c5754dc2ed8a541c3168d9b97c7db08c871
>
>
Could you please give me a filename as example for this ?
2013/9/16 Hannes Magnusson
> On Sun, Sep 15, 2013 at 3:27 PM, Chris Wright wrote:
> > I've pushed a couple of commits to clear the backlog here and
> > something that is definitely a problem with the checker is that it
> > considers var
On Sun, Sep 15, 2013 at 3:27 PM, Chris Wright wrote:
> I've pushed a couple of commits to clear the backlog here and
> something that is definitely a problem with the checker is that it
> considers varargs to have a missing initializer. Having the strict
> standards checker ignore any thing struct
I've pushed a couple of commits to clear the backlog here and
something that is definitely a problem with the checker is that it
considers varargs to have a missing initializer. Having the strict
standards checker ignore any thing structured like this:
...
...might cut down the size of the qu
OK great. I'll make a start on correcting the obvious ones and leave
any I'm not sure about alone, hopefully we can get the outstanding
list down to just these cases fairly quickly, which will quite
effectively separate the wheat from the chaff, and we can take another
look at the situation at that
That's a slightly different issue though.
In general functions with optional arguments have default value, or can be
faked with for example empty array or null. In those cases we should use
that for the initializer argument.
For others, I think we need to evaluate them on case by case bases what
On 10 September 2013 00:28, Hannes Magnusson wrote:
> IMO an optional argument should have a way to "skip it, using default value".
> its annoying when in userland you have to do
> if ($option) {
> foo($arg1, $arg2, $option);
> } else {
> foo($arg1, $arg2);
> }
>
> And I consider it a bug when I c
On Mon, Sep 9, 2013 at 3:37 PM, Chris Wright wrote:
> - What sort of timescale are we looking at for this (I'm not expecting
> dates but maybe weeks vs. months vs. years?)
We do things whenever we have time.
This change is even more tricky (although easier in some respect) then
the -> docbook5 b
Tangent:
On Mon, Sep 9, 2013 at 4:37 PM, Chris Wright wrote:
>
> I know the new MD-esque format is pretty much definitely going to
> happen...
Docs migration is a serious subject that needs more discussion if it's
going to happen.
Some of you may have noticed that I've been on a bit of a crusade to
nuke as many issues as possible in the OE - keep the patches list
empty, evaluate changes that are left in "work in progress" by users
who don't realise they need to create a patch (I know this is an issue
as I have previously don
14 matches
Mail list logo