Repository navigation
iconv codec-NG and Korean Codecs #38224
Description
| BPO | 710931 |
|---|---|
| Nosy | @loewis, @hyeshik |
| Files |
Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.
Show more details
GitHub fields:
assignee = 'https://github.com/loewis'
closed_at = <Date 2003-03-29.15:00:07.000>
created_at = <Date 2003-03-27.19:31:49.000>
labels = ['library']
title = 'iconv codec-NG and Korean Codecs'
updated_at = <Date 2003-03-29.15:00:07.000>
user = 'https://github.com/hyeshik'bugs.python.org fields:
activity = <Date 2003-03-29.15:00:07.000>
actor = 'loewis'
assignee = 'loewis'
closed = True
closed_date = None
closer = None
components = ['Library (Lib)']
creation = <Date 2003-03-27.19:31:49.000>
creator = 'hyeshik.chang'
dependencies = []
files = ['5133']
hgrepos = []
issue_num = 710931
keywords = ['patch']
message_count = 2.0
messages = ['43194', '43195']
nosy_count = 2.0
nosy_names = ['loewis', 'hyeshik.chang']
pr_nums = []
priority = 'normal'
resolution = 'rejected'
stage = None
status = 'closed'
superseder = None
type = None
url = 'https://bugs.python.org/issue710931'
versions = ['Python 2.3']
Activity
hyeshik commentedon Mar 27, 2003
This patch includes update for iconv_codec, new
sources for korean codecs and MultibyteCodec
supplemental library.
I splitted out common parts of codecs for usual
multibyte encodings into multibytecodec.c and this
iconv codec and the korean codecs are using it.
The korean codecs is only 58K in stripped i386 ELF and
62K in stripped i386 PECOFF binary and I think it's
small enough to be incorporated into python.
Files:
Lib/encodings/aliases.py
adds aliases for korean encodings and remove
comments that isn't true now.
Lib/encodings/cp949.py
Lib/encodings/euc_kr.py
codecs for korean encodings
Lib/encodings/iconv_codec.py
updated for new _iconv_codec implementation
Lib/test/test_ko_codecs.py
unit test for cp949, euc_kr codec
Lib/test/test_ko_codecs_mapping.py
unit test to test cp949 mapping
Lib/test/test_iconv_codec_euc_kr.py
another iconv_codec test unit. because non-unicode
multibyte encoding is required to test both of
iconv_codec and multibytecodec.
Lib/test/test_multibytecodec_support.py
common part for above test units
Modules/_iconv_codec.c
new implementation of _iconv_codec.
this resolves numerous problems that previous
implementation had. and iconv_codec has sane
StreamReader now! :)
Modules/_ko_codec.c
Modules/_ko_codec.h
korean codecs module
Modules/multibytecodec.c
Modules/multibytecodec.h
common multibyte codec supplement. I think that this
can be used for any usual multibyte encodings.
I'll submit Chinese Codecs in few days using this.
Tools/unicode/genmap_ko_codecs.py
code generator for _ko_codecs.h
loewis commentedon Mar 29, 2003
Logged In: YES
user_id=21627
Please submit an individual patch for each single bug fix or
new feature; it appears that this patch deals with
completely unrelated things. Therefore. I'm rejecting this
patch, encouraging you to submit new separate patches.
I have a few specific comments you may want to consider:
What is the rationale for adding an alias processing to
the iconv codecs?
It is unclear how you expect reuse of the
multibytecodec.c. Currently, this is incorporated into
_ko_codecs. How would this cooperate with other usages of
multibytecodecs? In particular, why is that needed in
iconv_codec?
"complete reimplementation" is insufficient reason to
accept a change. What specific problems does the old iconv
codec have, and how specifically have they been corrected?