Use the v2 parsing system for the OPTIONS response. It was not even using the v1 system, but some very early code that stylistically resembles what became v1 (svn_ra_serf__xml_state_t).
Use the v2 parsing system for the OPTIONS response. It was not even
using the v1 system, but some very early code that stylistically
resembles what became v1 (svn_ra_serf__xml_state_t).

This wipes out a bunch of code, along with making this more robust,
more pool-friendly, and easier to understand.

* subversion/libsvn_ra_serf/options.c:
  (options_state_e): add INITIAL to represent the context's initial state
  (options_state_list_t): removed. obsolete.
  (options_context_t): ACBUF, COLLECT_CDATA, STATE, FREE_STATE, DONE,
    PARSER_CTX are all removed as obsolete. added INNER_HANDLER and
    INNER_BATON to deal with our options response handler interposed
    before the expat handler.
  (D_, S_): helper namespace macros. S_ isn't actually used, but is
    there to match future precedent
  (options_ttable): new transition table for the OPTIONS body
  (push_state, pop_state, start_options, end_options, cdata_options):
    removed. obsolete.
  (options_closed): new v2 callback for capturing the activity
    collection set.
  (create_options_req): create a new xml context. stick that into an
    expat parsing handler. remove the v1-style xml parsing handler.
    interpose our custom handler in front of the expat handler.
  (svn_ra_serf__v2_get_youngest_revnum,
      svn_ra_serf__v1_get_activity_collection,
      svn_ra_serf__exchange_capabilities): wait on the handler's DONE
    field rather than the old-style DONE flag in the options context

jmckaskill/subversion,jmckaskill/subversion,jmckaskill/subversion,jmckaskill/subversion,jmckaskill/subversion,jmckaskill/subversion,jmckaskill/subversion,jmckaskill/subversion