Re: [PATCH] Converted Charset Fix

Paul P Komkoff Jr <[email protected]>
Newsgroups gmane.comp.mobile.kannel.devel
Organization Department of Fish & Wildlife
Message-ID <[email protected]>
Replying to Jonathan Houser:
>      So is the idea then to convert everything to UTF-8 within wap-appl 
> and then only serve UTF-8 to the phones (as they are required to support 
> at least UTF-8 if nothing else)?  This would also mean stripping the 
> encoding= section if it exists as UTF-8 is assumed if it's not present. 

... To convert everything to UTF8 or leave encoding as-is if phone
supports it. Strip preamble. Store somewhere in which encoding we have
xml now.
When parsing, do not mess up with encoding - just call xmlReadMemory
instead of xmlParseMemory and pass stored encoding to it. This change
will also allow us to specify some options like XML_PARSE_RECOVER.
When producing binary, (*wbxml)->character_set = our_stored_charset.

Summarizing that, we need:
1. Function to mess with encoding and strip preamble just after we got http reply. Call it
from return_reply. Remove copy&paste-programming artifacts from these.
2. Slight change in wml_compile(). Just remove all mess.
3. Slight change in (*wbxml)->character_set = line

>  If I'm correct on this, I'll start work on it.  As I'd said yesterday, 
> I really just want to get a patch committed so I can put this code into 
> production.

I was a little busy dealing with canadian embassy last week. And just
after I send this message I will be cut off internet until tomorrow
noon. After that, I can try to do described changes myself. But I
don't expect any difficulties.

-- 
Paul P 'Stingray' Komkoff Jr // http://stingr.net/key <- my pgp key
 This message represents the official view of the voices in my head
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.