Re: [PATCH 1/4] stringbuf: A string buffer implementation

!MAILaRCHIVE_VOTE_RePLACE
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: Linus Torvalds <torvalds@...>
Cc: <akpm@...>, <linux-kernel@...>, Matthew Wilcox <willy@...>
Date: Tuesday, October 23, 2007 - 10:30 pm

On Tue, Oct 23, 2007 at 04:43:53PM -0700, Linus Torvalds wrote:

I hadn't.  It's quite spookily similar in some ways.  There's a lot of
things you've added because you presumably have use for them in git that
could easily be added once we had users for them in the kernel.  There's
a few things stringbufs do that strbufs don't need to because they're in
userspace.

I can easily change the API at this point, so if renaming things will
make you happier, just say so.


It's a matter of taste ... some people prefer to use accessors for
everything, other people prefer to expose the underlying structure
directly.  I have to confess to being influenced by Java in 1998-99,
so possibly I tend to go for accessors too often.  Or possibly it's
just my experience of restructuring things in Linux where diffs would
be smaller if we only had used accessors for something.  In any case,
I don't care to fight about this; if you want it to be called buf,
then buf it shall be called, and I'll get rid of sb_string.

-- 
Intel are signing my paycheques ... these opinions are still mine
"Bill, look, we understand that you're interested in selling us this
operating system, but compare it to ours.  We can't possibly take such
a retrograde step."
-
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH 1/4] stringbuf: A string buffer implementation, Matthew Wilcox, (Tue Oct 23, 5:12 pm)
Re: [PATCH 1/4] stringbuf: A string buffer implementation, Denys Vlasenko, (Tue Oct 30, 11:26 am)
Re: [PATCH 1/4] stringbuf: A string buffer implementation, Florian Weimer, (Wed Oct 24, 9:21 am)
Re: [PATCH 1/4] stringbuf: A string buffer implementation, Matthew Wilcox, (Wed Oct 24, 10:02 am)
Re: [PATCH 1/4] stringbuf: A string buffer implementation, Matthew Wilcox, (Tue Oct 23, 10:35 pm)
Re: [PATCH 1/4] stringbuf: A string buffer implementation, Eric St-Laurent, (Tue Oct 23, 10:48 pm)
Re: [PATCH 1/4] stringbuf: A string buffer implementation, Linus Torvalds, (Tue Oct 23, 7:43 pm)
Re: [PATCH 1/4] stringbuf: A string buffer implementation, Matthew Wilcox, (Tue Oct 23, 10:30 pm)
Re: [PATCH 1/4] stringbuf: A string buffer implementation, Andrew Morton, (Tue Oct 23, 10:45 pm)
Re: [PATCH 1/4] stringbuf: A string buffer implementation, Eric St-Laurent, (Tue Oct 23, 10:19 pm)
Re: [PATCH 1/4] stringbuf: A string buffer implementation, Matthew Wilcox, (Tue Oct 23, 9:49 pm)
Re: [PATCH 1/4] stringbuf: A string buffer implementation, Matt Mackall, (Wed Oct 24, 11:20 am)
Re: [PATCH 1/4] stringbuf: A string buffer implementation, Matthew Wilcox, (Wed Oct 24, 11:30 am)
[PATCH 2/4] isdn: Use stringbuf, Matthew Wilcox, (Tue Oct 23, 5:12 pm)
[PATCH 3/4] sound: Use stringbuf, Matthew Wilcox, (Tue Oct 23, 5:12 pm)
Re: [PATCH 3/4] sound: Use stringbuf, Takashi Iwai, (Wed Oct 24, 10:18 am)
Re: [PATCH 3/4] sound: Use stringbuf, Matthew Wilcox, (Wed Oct 24, 12:01 pm)
Re: [PATCH 3/4] sound: Use stringbuf, Takashi Iwai, (Wed Oct 24, 10:50 am)
[PATCH 4/4] partitions: Fix non-atomic printk, Matthew Wilcox, (Tue Oct 23, 5:12 pm)