Re: [PATCH] It's UTF-8

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Jan Engelhardt <[email protected]> writes:

>>Signed-off-by: Alexey Dobriyan <[email protected]>
>
> I'd say ACK. However,
>
>> iocharset=name	Character set to use for converting from Unicode to
>> 		ASCII.  The default is to do no conversion.  Use
>>-		iocharset=utf8 for UTF8 translations.  This requires
>>+		iocharset=utf8 for UTF-8 translations.  This requires
>> 		CONFIG_NLS_UTF8 to be set in the kernel .config file.
>
> If you are really nitpicky about the "-", then it should also be 
> "iocharset=utf-8" (and whereever else). Or what's the real purpose of 
> adding the dashes in only half of the places, then?

The patch only changes documentation/comments.  Changing other things
would break compatibility, and that's usually not a good idea for
cosmetic changes.

-- 
Måns Rullgård
[email protected]

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux