Message38132
| Author |
loewis |
| Recipients |
| Date |
2001年11月09日.21:21:09 |
| SpamBayes Score |
| Marked as misclassified |
| Message-id |
| In-reply-to |
| Content |
Logged In: YES
user_id=21627
Even though I think this patch is correct in principle, I
see a few problems with it:
1. Since it doesn't fix a bug, it probably cannot go into 2.2.
2. There is no autoconf test for mbtowc. You should test
this in configure, and then conditionalize your code on
HAVE_MBTOWC.
3. There is too much code duplication. Try to find a
solution which special-cases the escape codes (\something)
only once. For example, you may implement a trivial mbtowc
redefinition if mbtowc is not available, and then use mbtowc
always. |
|
History
|
|---|
| Date |
User |
Action |
Args |
| 2007年08月23日 15:08:55 | admin | link | issue479898 messages |
| 2007年08月23日 15:08:55 | admin | create |
|