Re: [PATCH 3/7] IMFS: Simplify IMFS_symlink()

2015-02-10 Thread Sebastian Huber
On 09/02/15 22:28, Gedare Bloom wrote: Nit: should the counted string functions be used? No, the allocation part takes care that we have enough space in the buffer. On Mon, Feb 9, 2015 at 3:35 PM, Sebastian Huber wrote: --- cpukit/libfs/src/imfs/imfs_symlink.c | 33 +++--

Re: [PATCH 3/7] IMFS: Simplify IMFS_symlink()

2015-02-09 Thread Gedare Bloom
Nit: should the counted string functions be used? On Mon, Feb 9, 2015 at 3:35 PM, Sebastian Huber wrote: > --- > cpukit/libfs/src/imfs/imfs_symlink.c | 33 +++-- > 1 file changed, 7 insertions(+), 26 deletions(-) > > diff --git a/cpukit/libfs/src/imfs/imfs_symlink.c

[PATCH 3/7] IMFS: Simplify IMFS_symlink()

2015-02-09 Thread Sebastian Huber
--- cpukit/libfs/src/imfs/imfs_symlink.c | 33 +++-- 1 file changed, 7 insertions(+), 26 deletions(-) diff --git a/cpukit/libfs/src/imfs/imfs_symlink.c b/cpukit/libfs/src/imfs/imfs_symlink.c index 2ddf99c..0f06446 100644 --- a/cpukit/libfs/src/imfs/imfs_symlink.c +++