httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "William A. Rowe, Jr." <wr...@rowe-clan.net>
Subject Re: [PATCH] Add AP_INIT_TAKE_ARGV
Date Wed, 04 May 2005 20:37:00 GMT
At 02:34 PM 5/4/2005, Paul Querna wrote:
>Attached is a patch that adds a new configuration command type.
>
>It uses the traditional argv/argc. I grew tired of having to make
>commands more complicated by using RAW_ARGS, when there is no need.
>
>The patch includes changing 'IndexOptions' to use the new
>AP_INIT_TAKE_ARGV, instead of RAW_ARGS.  It also adds a minor MMN bump
>for the new interface.

One goal is to be able to plug in an alternative xml-conf style
parser.  Your proposal mapps much more nicely for xml configs;

  <olddirective>
    <args>foo bar bash</args>
  </olddirective>

  <newdirective>
    <arg>foo</arg>
    <arg>bar</arg>
    <arg>bash</arg>
  </newdirective>

so definitely ++1 in theory, without reviewing the code yet.  


Mime
View raw message