patchgnokii - Patches: patch #10001, IRA encoding

 
 

You are not allowed to post comments on this tracker with your current authentication level.

patch #10001: IRA encoding

Submitter:  Raphaël Droz <drzraf>
Submitted:  Fri 11 Dec 2020 03:11:35 AM UTC
   
 
Category:  gnokii Priority:  3 - Low
Status:  None Privacy:  Public
Assigned to:  None Open/Closed:  Open

Sun 13 Dec 2020 09:51:31 PM UTC, comment #1: 

Why you added AT_CHAR_IRA only to at_encode() and not to at_decode()?

How do you handle the characters that are different from ASCII?

Does iconv converts your strings correctly? It supports several ISO646-* charsets that IIUC correspond to the national variants of IRA

Do you have a test string that we can add somewhere, at least in the commit message? I think our testsuite isn't flexible enough.
https://git.savannah.nongnu.org/cgit/gnokii.git/tree/testsuite

TNX

Daniele Forsi <dforsi>
Group administrator
Fri 11 Dec 2020 03:11:35 AM UTC, original submission:  

Found that encoding as the only one supported by an "MTT-Mini" feature phone.

Raphaël Droz <drzraf>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attached Files
file #50437:  ira-encoding.patch added by drzraf (2KiB - text/x-patch)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by dforsi (Posted a comment)
  • -email is unavailable- added by drzraf (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

     

    Follows 1 latest change.

    Date Changed by Updated Field Previous Value => Replaced by
    2020-12-11 drzraf Attached File- Added ira-encoding.patch, #50437

    Back to the top

    Powered by Savane 3.13-d3ae.
    Corresponding source code