Re: Small cleanup for utilities/dbootstrap/release_notes.c

2002-03-13 Thread Petter Reinholdtsen
[Petter Reinholdtsen] It works. Here is a new patch to be able to test displaying release notes in other languages. Index: release_notes.c === RCS file: /cvs/debian-boot/boot-floppies/utilities/dbootstrap/release_notes.c,v

Re: Small cleanup for utilities/dbootstrap/release_notes.c

2002-03-13 Thread Junichi Uekawa
Petter Reinholdtsen [EMAIL PROTECTED] cum veritate scripsit: Any hopes of getting the two cleanup patches applied? Your small cleanup isn't very small. Has anybody inspected the code / checked it ? regards, junichi -- [EMAIL PROTECTED] : Junichi Uekawa

Re: Small cleanup for utilities/dbootstrap/release_notes.c

2002-03-13 Thread Matt Kraai
On Wed, Mar 13, 2002 at 10:36:39PM +0900, Junichi Uekawa wrote: Petter Reinholdtsen [EMAIL PROTECTED] cum veritate scripsit: Any hopes of getting the two cleanup patches applied? Your small cleanup isn't very small. Has anybody inspected the code / checked it ? I've inspected, tested,

Re: Small cleanup for utilities/dbootstrap/release_notes.c

2002-03-11 Thread Petter Reinholdtsen
[Petter Reinholdtsen] BTW: The test code is unable to test other languages. Should this be fixed? If it should be fixed, here is a patch to fix it. release_notes_test seem to be broken for all non-english texts. I've tested de and da. The width calculation fails. Index: release_notes.c

Re: Small cleanup for utilities/dbootstrap/release_notes.c

2002-03-11 Thread Junichi Uekawa
Petter Reinholdtsen [EMAIL PROTECTED] cum veritate scripsit: BTW: The test code is unable to test other languages. Should this be fixed? If it should be fixed, here is a patch to fix it. release_notes_test seem to be broken for all non-english texts. I've tested de and da. The

Re: Small cleanup for utilities/dbootstrap/release_notes.c

2002-03-11 Thread Petter Reinholdtsen
[Junichi Uekawa] Of course, there is no setlocale call. Ah, of course. I'm not sure why, but I was sure that was taken care of in some library somewhere. :-) If it doesn't work, you've found a real bug. It works. Here is a new patch to be able to test displaying release notes in other