Hi Philip,
Philip Martin wrote:
>"Peter N. Lundblad" <peter@famlundblad.se> writes:
>
>
>
>>Ouch! Putting vars of those sizes on the stack isn't very friendly. Just
>>allocate it out of a pool instead.
>>
>>(Saying this without having investigated in detail, but I can't imagine it
>>should be a problem.)
>>
>>
>
>Look at this:
>
>
This is the only case, where a pool is not available.
>svn_error_t *
>svn_subst_translate_stream (svn_stream_t *s, /* src stream */
> svn_stream_t *d, /* dst stream */
> const char *eol_str,
> svn_boolean_t repair,
> const svn_subst_keywords_t *keywords,
> svn_boolean_t expand)
>{
> char buf[SVN_STREAM_CHUNK_SIZE + 1];
> const char *p, *interesting;
>
>There is no readily available pool, and neither svn_stream_t nor
>svn_subst_keywords_t provides one. There is a void* baton in
>svn_stream_t but svn_subst_translate_stream can't really make any
>assumptions about it.
>
>
All callers of this function have a pool available. If I could extend
the interface by a pool parameter, I could change that case too.
Patrick
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Sun May 8 14:55:21 2005