Re: [PATCH] Add svnrdump

Hi Daniel,

Daniel Shahaf writes:
> > > + /* Use a temporary file to measure the text-content-length */
> > > + apr_err = apr_temp_dir_get(&tempdir, hb->pool);
>
> svn_io_temp_dir()

Fixed.

> > > + if (apr_err != APR_SUCCESS)
> > > + SVN_ERR(svn_error_wrap_apr(apr_err, NULL));
> > > +
> > > + hb->temp_filepath = apr_psprintf(eb->pool, "%s/svn-fe-XXXXXX",
> > > tempdir);
> >
>
> os.path.join()
>
> Err, I mean, svn_dirent_join().

Fixed. Thanks :)

-- Ram
Received on 2010-07-09 15:42:18 CEST