BC break in headers_list

Hi Stas,

Hi Anatol!

Your recent fixes to headers_list() -
55cefb2814bde5815a92e8820fff45e037fa8d4f and
b5d3c5ca8dee6303498849448e3574cc3642eeea - broke head.phpt test and also
are BC-breaking since previously headers_list() always returned an array
(empty one if there are no headers), now it returns false when there's
no headers. As the manual also says it returns array, I would rather delete
the RETURN_FALSE altogether. But if there's a good reason to return false,
the tests should be fixed and the notes in the UPGRADING and the manual
should be added. I think just reverting to the previous behavior is
better. --
Thanks for the headsup, I must have missed that test. But despite that,
the condition there how it was (and is still everywhere, even in 5.4) is
erroneous. I mean "if (!&SG(sapi_headers).headers)" is always false, as
the sapi_headers member is a non-pointer, so taking its address is always
true.

So the intention was to correct that part to do what it actually looks
like. But since it was unlikely working and headers_list() was never
returning false and is documented to return array - it's probably better
to just remove those three lines in 5.5 onwards (might be cut out by
optimizer anyway). Anyway agreed, no good reasons to force it to do what
it never did. I haven't dig deeper, maybe it was worky back then in early
PHP5 if "headers" were a pointer, but it doesn't behave as expected at
least since PHP5.4. I'm to remove that check it 5.5+ then.

Regards

Anatol