Martin Karlgren wrote: >> On 20 Sep 2016, at 09:59 , Stephen R. van den Berg <s...@cuci.nl> wrote:
>> Stephen R. van den Berg wrote: >>> Arne Goedeke wrote: >>>> Yes, I agree that having the query string parsed is be very useful. >>>> There is some other logic in parse_variables() which we do not want, so >>>> we would have to make sure that none of that has any side effects. I >>>> would propose we explicitly call only the query decode and >>>> flatten_headers(). >>> I'll commit something to that effect. >> Committed, please review (both on 8.0 and 8.1). >Commit 7ce9c4ae introduced an incompatible API change in >HTTP.Server.Request.parse_variables. We have an inheriting class that calls >parse_variables() without arguments, which now fails to compile. Should the >new parameter ???short??? be declared ???void|int??? ? void|int will do. Fixed and committed to 8.0 and 8.1 just now. -- Stephen.
