Re: [PATCH] Make working copies read-only.

On Thu, Nov 29, 2001 at 10:57:00PM -0500, Kevin Pilch-Bisson wrote:
>...
> > > +++ ./subversion/libsvn_wc/get_editor.c Tue Nov 27 09:42:39 2001
> > > @@ -1155,6 +1155,9 @@
> > > tmp_txtb,
> > > SVN_WC__LOG_ATTR_DEST,
> > > txtb,
> > > + SVN_WC__LOG_ATTR_PERM,
> > > + svn_stringbuf_create
> > > + (SVN_WC__LOG_PERM_READONLY, fb->pool),
> > > NULL);
> > >
> > > if ( (! is_locally_modified)
> > > @@ -1171,6 +1174,9 @@
> > > txtb,
> > > SVN_WC__LOG_ATTR_DEST,
> > > fb->name,
> > > + SVN_WC__LOG_ATTR_PERM,
> > > + svn_stringbuf_create
> > > + (SVN_WC__LOG_PERM_DEFAULT, fb->pool),
> > > NULL);
> > > }
> > >
> > > @@ -1198,6 +1204,10 @@
> > > txtb,
> > > SVN_WC__LOG_ATTR_DEST,
> > > fb->name,
> > > + SVN_WC__LOG_ATTR_PERM,
> > > + svn_stringbuf_create
> > > + (SVN_WC__LOG_PERM_DEFAULT,
> > > + fb->pool),
> > > NULL);
> > > }
> > > else /* working file exists */
> > > @@ -1411,6 +1421,9 @@
> > > fb->name,
> > > SVN_WC__LOG_ATTR_DEST,
> > > renamed_basename,
> > > + SVN_WC__LOG_ATTR_PERM,
> > > + svn_stringbuf_create
> > > + (SVN_WC__LOG_PERM_SRC, fb->pool),
> > > NULL);
> > >
> > > /* Copy the new file out into working area. */
> > > @@ -1422,6 +1435,9 @@
> > > txtb,
> > > SVN_WC__LOG_ATTR_DEST,
> > > fb->name,
> > > + SVN_WC__LOG_ATTR_PERM,
> > > + svn_stringbuf_create
> > > + (SVN_WC__LOG_PERM_DEFAULT, fb->pool),
> > > NULL);

Holy crap, that is some major suckage. Allocate a heap buffer so that we can
put a constant string into the output XML? Blearg.

I guess the one saving grace is that it isn't within a loop, and fb->pool is
about to be destroyed. But sheesh... what convolutions for an otherwise
simple task :-(

Cheers,
-g

-- 
Greg Stein, http://www.lyra.org/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org