Hi Stephen, > 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” ? Regards, /Marty
