[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: CVS update: subversion/subversion/include svn_delta.h

From: Ben Collins-Sussman <sussman_at_newton.ch.collab.net>
Date: 2001-03-02 00:17:36 CET

Greg Stein <gstein@lyra.org> writes:

> > * I notice that you *statically* declare and initialize a window
> > struct, and even an svn_string_t. Isn't this a dangerous,
> > non-robust practice? :)
>
> Why would it be dangerous? Nobody can write outside of those bounds, and
> nobody should be attempting to free() the pointer.

Oh, I just meant that if we ever change the definition of a struct,
your static initialization would more likely break, compared to
palloc'ing it.

But who cares. :)
Received on Sat Oct 21 14:36:24 2006

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.