From: konrad wilk <konrad.wilk@oracle.com>
To: Jan Beulich <jbeulich@suse.com>
Cc: xen-devel@lists.xenproject.org
Subject: Re: pre_parse and strcmp
Date: Fri, 03 Oct 2014 12:22:43 -0400 [thread overview]
Message-ID: <542ECD53.7090807@oracle.com> (raw)
In-Reply-To: <542EDA3B02000078000BE950@mail.emea.novell.com>
On 10/3/2014 12:17 PM, Jan Beulich wrote:
>>>> Konrad Rzeszutek Wilk <konrad.wilk@oracle.com> 10/02/14 11:15 PM >>>
>> Hit sent to fast [was going to include a patch in this email
>> once I had completed this]
>>
>> My thinking is that the best solution is to have a similar to 'pre_parse'
>> function that would convert the in memory buffer from UTF-16 to a normal
>> ascii type one.
>>
>> And hook it up in pre-parse to fix this up.
>
> I certainly don't mind a patch to deal with UTF-16 config files so (in fact already
> when I originally coded it I considered this would be a good future enhancement).
> I'm not, however, convinced that simply converting back to ASCII is the proper
> solution here. Instead, if we want to allow UTF-16 config files, we should do the
> conversion the other way around.
OK. That will take some time to cobble up.
>
> And then of course there is the problem of detection: The example you gave didn't
> make clear whether the file was properly starting with a BOM, yet if it doesn't
> telling ASCII from UTF-16 is guesswork.
Ah, so that is what the odd character at the start was (BOM)! Yes, the
file is very much that type.
>
> Jan
>
>
next prev parent reply other threads:[~2014-10-03 16:22 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-10-02 21:09 pre_parse and strcmp Konrad Rzeszutek Wilk
2014-10-02 21:15 ` Konrad Rzeszutek Wilk
2014-10-03 16:17 ` Jan Beulich
2014-10-03 16:22 ` konrad wilk [this message]
2014-10-04 14:57 ` Don Slutz
2014-10-06 8:11 ` Jan Beulich
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=542ECD53.7090807@oracle.com \
--to=konrad.wilk@oracle.com \
--cc=jbeulich@suse.com \
--cc=xen-devel@lists.xenproject.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.