[Patches] [ python-Patches-626548 ] Support Hangul Syllable names
noreply@sourceforge.net
noreply@sourceforge.net
Wed, 23 Oct 2002 04:31:26 -0700
Patches item #626548, was opened at 2002-10-21 23:11
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=626548&group_id=5470
Category: None
Group: None
Status: Open
Resolution: None
Priority: 5
Submitted By: Martin v. Löwis (loewis)
Assigned to: M.-A. Lemburg (lemburg)
Summary: Support Hangul Syllable names
Initial Comment:
This patch implements section 25.2 of ISO 10646
(Character names and annotations for Hangul syllables).
----------------------------------------------------------------------
>Comment By: Martin v. Löwis (loewis)
Date: 2002-10-23 13:31
Message:
Logged In: YES
user_id=21627
Can you please elaborate this position? Is it not important
to follow the established and agreed style guide?
----------------------------------------------------------------------
Comment By: M.-A. Lemburg (lemburg)
Date: 2002-10-23 13:02
Message:
Logged In: YES
user_id=38388
Apart from that the patch looks ok. Do you have some docs
to go with it ?
----------------------------------------------------------------------
Comment By: M.-A. Lemburg (lemburg)
Date: 2002-10-23 13:00
Message:
Logged In: YES
user_id=38388
No, I'd rather leave things as they are w/r to indentation.
Thanks.
----------------------------------------------------------------------
Comment By: Martin v. Löwis (loewis)
Date: 2002-10-23 12:42
Message:
Logged In: YES
user_id=21627
Perhaps the entire file should be formatted to conform with
PEP 7 (single-tab indents, where a tab is worth 8 spaces).
Should I submit a separate patch for this reformatting?
----------------------------------------------------------------------
Comment By: M.-A. Lemburg (lemburg)
Date: 2002-10-23 12:32
Message:
Logged In: YES
user_id=38388
One more minor nit: the indentation in the C file is 4
chars, please
reindent your code accordingly.
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=626548&group_id=5470