On Fri, Aug 20, 2010 at 13:58, <julianf...@apache.org> wrote:
>...
> +++ subversion/trunk/subversion/libsvn_ra_neon/util.c Fri Aug 20 17:58:10 2010
>...
> -static ne_xml_parser *
> +/* Create a status parser attached to the request REQ. Detected errors
> + will be returned there. */
> +static void
> multistatus_parser_create(svn_ra_neon__request_t *req)
> {
> multistatus_baton_t *b = apr_pcalloc(req->pool, sizeof(*b));
> - ne_xml_parser *multistatus_parser =
> - svn_ra_neon__xml_parser_create(req, ne_accept_207,
> - start_207_element,
> - svn_ra_neon__xml_collect_cdata,
> - end_207_element, b);
> +
> + svn_ra_neon__xml_parser_create(req, ne_accept_207,
Woah. You should have a (void) cast in there and (or?) a comment
noting that the return value is not used. Otherwise, it looks like a
programming error.
>...
Cheers,
-g
