All of lore.kernel.org
 help / color / mirror / Atom feed
From: Denis Kenzior <denkenz@gmail.com>
To: ofono@ofono.org
Subject: Re: [PATCH 2/4] Enable alphabets in smsutils
Date: Fri, 03 Sep 2010 10:08:10 -0500	[thread overview]
Message-ID: <4C810F5A.2050806@gmail.com> (raw)
In-Reply-To: <AANLkTi=DWN54mvFZzn=odY9soF6h4hLVyQt30T0CvBSO@mail.gmail.com>

[-- Attachment #1: Type: text/plain, Size: 1390 bytes --]

Hi Pekka,

On 09/03/2010 09:40 AM, Pekka Pessi wrote:
> Morjes Aki,
> 
> 2010/9/2 Aki Niemi <aki.niemi@nokia.com>:
>> +       case SMS_ALPHABET_TURKISH:
>> +               gsm_encoded = convert_utf8_to_gsm_with_lang(utf8, -1, NULL,
>> +                                                       &written, 0,
>> +                                                       GSM_DIALECT_TURKISH,
>> +                                                       GSM_DIALECT_TURKISH);
> 
> If you look at the tables,  the idea is normally to use either locking
> Turkish and default single, or default locking and Turkish single
> shift, never both.

The spec also says: "however it is possible to use both single shift and
locking shift with the corresponding tables in a single message."

So how exactly do we know which combination to try? Is it language
dependent?

> 
> Also, if it is possible to fit the message to one segment only using
> national single shift table, single shift should be used (if receiver
> does not support national variants, the result is much less garbled).
> 

So what you're saying is that we should try these three combinations:

default locking, default single shift
default locking, language single shift
language locking shift, language single shift

Trying language locking, default single shift does not seem useful.

Regards,
-Denis

  reply	other threads:[~2010-09-03 15:08 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-09-02  7:52 [PATCH 1/4] Add so called reduced charset support to SMS Aki Niemi
2010-09-02  7:52 ` [PATCH 2/4] Enable alphabets in smsutils Aki Niemi
2010-09-02 16:55   ` Denis Kenzior
2010-09-03 12:15     ` Aki Niemi
2010-09-03 14:05       ` Denis Kenzior
2010-09-03 14:12         ` Aki Niemi
2010-09-03 14:40   ` Pekka Pessi
2010-09-03 15:08     ` Denis Kenzior [this message]
2010-09-03 17:09     ` Aki Niemi
2010-09-02  7:52 ` [PATCH 3/4] Add alphabet support to SMS atom Aki Niemi
2010-09-02 17:00   ` Denis Kenzior
2010-09-02  7:52 ` [PATCH 4/4] Add documentation for the Alphabet property Aki Niemi
2010-09-02 17:01   ` Denis Kenzior
2010-09-02 18:19     ` Marcel Holtmann
2010-09-02 19:08       ` Aki Niemi
2010-09-02 16:43 ` [PATCH 1/4] Add so called reduced charset support to SMS Denis Kenzior

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=4C810F5A.2050806@gmail.com \
    --to=denkenz@gmail.com \
    --cc=ofono@ofono.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.