Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 26 Sep 2003 17:30:55 +0700
From:      Boris Popov <bp@freebsd.org>
To:        "R. Imura" <imura@ryu16.org>
Cc:        freebsd-i18n@freebsd.org
Subject:   Re: [Patch] cd9660/msdosfs/ntfs/libiconv mutli-byte extention
Message-ID:  <20030926103055.GD48438@vertex.kz>
In-Reply-To: <20030922170903.GA43754@ryu16.org>
References:  <20030922170903.GA43754@ryu16.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Tue, Sep 23, 2003 at 02:09:03AM +0900, R. Imura wrote:
> 
> I made a patch for multi-byte character extension
> of three file systems with enhanced LIBICONV in the
> kernel. File systems are cd9660(joliet part), msdosfs,
> and ntfs.

	Great work, I should say!  The patch itself looks good.  There is
only one minor note: function iconv_vfs_refcount() looks like misplaced one.
It doesn't really belong to the iconv module and pollutes it with mount.h.
May be getting this function into one of vfs_*.c files would be a better
idea, although I'm don't have strong opinion on that.
> 
> New Kernel Options (Kernel Modules)
>  - CD9660_ICONV (cd9660_iconv.ko)
>  - MSDOSFS_ICONV (msdosfs_iconv.ko)
>  - NTFS_ICONV (ntfs_iconv.ko)

	*_iconv.ko modules aren't strictly necessary and at some point
they could be integrated into corresponding fs modules. libiconv.ko module
isn't that large to significantly bloat install floppies.
> 
> Any comments? And is it possible to apply this
> patch to upcoming FreeBSD-5.2?

	I think yes. In addition, it is a good candidate for MFC into RELENG_4.
> 
> P.S.
> Though it may be very small thing,

	No this is great thing, thanks for your work!

-- 
Boris Popov
http://rbp.euro.ru



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20030926103055.GD48438>