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

Re: [patch] First stab at text deltas

From: Branko Čibej <branko.cibej_at_hermes.si>
Date: 2000-08-10 18:45:42 CEST

Jim Blandy wrote:
>
> The code looks good so far.
>
> It's fine with me to move svn_tdelta__winfo_t's ops_space into
> svn_delta_window_t. That field is useful to anyone wanting to operate
> on windows, which seems like something lots of code might need.

That'd be better, yes.

> Whenever I'm working with an array which might have more space
> allocated than used, I use `num_' to indicate the number of elements
> actually in use, and `_size' to indicate the allocated size (to
> suggest `sizeof'). If that convention appeals to you, you could call
> the field ops_size. But maybe num_ vs. _size is a weak distinction,
> making a poor convention.

I try to avoid using "size" for arrays when I mean length. Sometimes I
use _count and _capacity. Anyway, you're the author of the interface,
so I'll adapt to your style.
 
> I believe that svn_tdelta__winfo_t's data_offset is always equal to
> the window's new->len. If you eliminate that variable, you can get
> rid of svn_tdelta__winfo_t altogether. That would simplify your code

Indeed it is, but that would mean exposing the implementation details
of svn_string_t. If that's O.K., I'll do that an remove svn_tdelta__winfo_t.

>
> a bit. But eventually you'll have hash tables and other state anyway,
> so it may not be important.

In fact, I already _do_ have hash tables and other state ...
I've just now finished the core vdelta driver, along with the dumb
windowing method. I need to clean up the code a bit, so I'll have a
patch ready sometime tomorrow.

Then I'm off for a week.

> I sense that you have a lispy background. :)

Not really. I first saw this kind of syntax a long time ago when
I was playing around with TeX's virtual font descriptions, and hadn't
even heard of lisp. Sorry to disappoint you :-)
(Today, of course, I regularly hack at my .emacs file).

    Brane

-- 
Branko Čibej                 <branko.cibej@hermes.si>
HERMES SoftLab, Litijska 51, 1000 Ljubljana, Slovenia
voice: (+386 1) 586 53 49     fax: (+386 1) 586 52 70
Received on Sat Oct 21 14:36:06 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.