From edi at agharta.de Mon Feb 19 07:51:39 2007 From: edi at agharta.de (Edi Weitz) Date: Mon, 19 Feb 2007 08:51:39 +0100 Subject: [flexi-streams-devel] New release 0.10.3 Message-ID: ChangeLog: Version 0.10.3 2007-02-19 Fixed bug in UTF-16 output (patch by Stelian Ionescu) Fixed *SUBSTITUTION-CHAR* example in docs Download: http://weitz.de/files/flexi-streams.tar.gz Cheers, Edi. From edi at agharta.de Thu Feb 22 18:09:59 2007 From: edi at agharta.de (Edi Weitz) Date: Thu, 22 Feb 2007 19:09:59 +0100 Subject: [flexi-streams-devel] Darcs repositories Message-ID: [My apologies if you get this more than once.] Several people have asked for Darcs repositories of my software. These do exists now: http://common-lisp.net/~loliveira/ediware/ Special thanks to Lu?s Oliveira who made this possible and who maintains the repositories. Cheers, Edi. From vodonosov at mail.ru Sat Feb 24 22:13:52 2007 From: vodonosov at mail.ru (Anton Vodonosov) Date: Sun, 25 Feb 2007 00:13:52 +0200 Subject: [flexi-streams-devel] refactoring *provide-use-value-restart* In-Reply-To: References: Message-ID: <45E0B8A0.1020601@mail.ru> Hi! Edi, what do you think about following refactoring. Remove *provide-use-value-restart* variable, but save performance. To avoid verbose description, I've attached a patch. It is only a sketch of how it can be done, I event haven't compiled it. Thus public interface of the library may be simplified, at the cost of more complex responsibilities of char-readers defined using define-char-reader macro: they should use recover-from-encoding-error. By the way. May be define-char-code-reader is more accurate name for the macro than define-cahr-reader Regards, -Anton -------------- next part -------------- An embedded and charset-unspecified text was scrubbed... Name: input.lisp.diff URL: From vodonosov at mail.ru Sat Feb 24 22:30:58 2007 From: vodonosov at mail.ru (Anton Vodonosov) Date: Sun, 25 Feb 2007 00:30:58 +0200 Subject: [flexi-streams-devel] refactoring *provide-use-value-restart* In-Reply-To: <45E0B8A0.1020601@mail.ru> References: <45E0B8A0.1020601@mail.ru> Message-ID: <45E0BCA2.5020208@mail.ru> Sorry, patch in previouse message contains few copy-pasted lines that I forgot to remove. Right version is attached. -Anton Anton Vodonosov: > Hi! > > Edi, what do you think about following refactoring. > > Remove *provide-use-value-restart* variable, but save > performance. > > To avoid verbose description, I've attached a patch. > It is only a sketch of how it can be done, I event > haven't compiled it. > > Thus public interface of the library may be simplified, > at the cost of more complex responsibilities of char-readers > defined using define-char-reader macro: they should use > recover-from-encoding-error. > > By the way. May be define-char-code-reader is more accurate > name for the macro than define-cahr-reader > > Regards, > -Anton > > -------------- next part -------------- An embedded and charset-unspecified text was scrubbed... Name: input.lisp.diff URL: From vodonosov at mail.ru Sat Feb 24 22:50:44 2007 From: vodonosov at mail.ru (Anton Vodonosov) Date: Sun, 25 Feb 2007 00:50:44 +0200 Subject: [flexi-streams-devel] refactoring *provide-use-value-restart* In-Reply-To: <45E0BCA2.5020208@mail.ru> References: <45E0B8A0.1020601@mail.ru> <45E0BCA2.5020208@mail.ru> Message-ID: <45E0C144.7070003@mail.ru> Rightest one attached here - using (char-code *substitution-char*) instead of *substitution-char* itself. Sorry again, -Anton Anton Vodonosov: > Sorry, patch in previouse message contains few copy-pasted > lines that I forgot to remove. Right version is attached. > > -Anton > > Anton Vodonosov: >> Hi! >> >> Edi, what do you think about following refactoring. >> >> Remove *provide-use-value-restart* variable, but save >> performance. >> >> To avoid verbose description, I've attached a patch. >> It is only a sketch of how it can be done, I event >> haven't compiled it. >> >> Thus public interface of the library may be simplified, >> at the cost of more complex responsibilities of char-readers >> defined using define-char-reader macro: they should use >> recover-from-encoding-error. >> >> By the way. May be define-char-code-reader is more accurate >> name for the macro than define-cahr-reader >> >> Regards, >> -Anton >> >> -------------- next part -------------- An embedded and charset-unspecified text was scrubbed... Name: input.lisp.diff URL: From vodonosov at mail.ru Sun Feb 25 13:59:02 2007 From: vodonosov at mail.ru (Anton Vodonosov) Date: Sun, 25 Feb 2007 15:59:02 +0200 Subject: [flexi-streams-devel] refactoring *provide-use-value-restart* In-Reply-To: <45E0C144.7070003@mail.ru> References: <45E0B8A0.1020601@mail.ru> <45E0BCA2.5020208@mail.ru> <45E0C144.7070003@mail.ru> Message-ID: <45E19626.5000800@mail.ru> We also can get rid of *use-replacement-char*, as in the attached pseudo patch. Regards, -Anton Anton Vodonosov: > Rightest one attached here - using > (char-code *substitution-char*) instead of > *substitution-char* itself. > > Sorry again, > -Anton > > Anton Vodonosov: >> Sorry, patch in previouse message contains few copy-pasted >> lines that I forgot to remove. Right version is attached. >> >> -Anton >> >> Anton Vodonosov: >>> Hi! >>> >>> Edi, what do you think about following refactoring. >>> >>> Remove *provide-use-value-restart* variable, but save >>> performance. >>> >>> To avoid verbose description, I've attached a patch. >>> It is only a sketch of how it can be done, I event >>> haven't compiled it. >>> >>> Thus public interface of the library may be simplified, >>> at the cost of more complex responsibilities of char-readers >>> defined using define-char-reader macro: they should use >>> recover-from-encoding-error. >>> >>> By the way. May be define-char-code-reader is more accurate >>> name for the macro than define-cahr-reader >>> >>> Regards, >>> -Anton >>> >>> -------------- next part -------------- An embedded and charset-unspecified text was scrubbed... Name: input.lisp.diff URL: From edi at agharta.de Sun Feb 25 20:52:35 2007 From: edi at agharta.de (Edi Weitz) Date: Sun, 25 Feb 2007 21:52:35 +0100 Subject: [flexi-streams-devel] refactoring *provide-use-value-restart* In-Reply-To: <45E0B8A0.1020601@mail.ru> (Anton Vodonosov's message of "Sun, 25 Feb 2007 00:13:52 +0200") References: <45E0B8A0.1020601@mail.ru> Message-ID: On Sun, 25 Feb 2007 00:13:52 +0200, Anton Vodonosov wrote: > Edi, what do you think about following refactoring. > > Remove *provide-use-value-restart* variable, but save performance. > > To avoid verbose description, I've attached a patch. It is only a > sketch of how it can be done, I event haven't compiled it. > > Thus public interface of the library may be simplified, at the cost > of more complex responsibilities of char-readers defined using > define-char-reader macro: they should use > recover-from-encoding-error. > > By the way. May be define-char-code-reader is more accurate name for > the macro than define-cahr-reader Hi Anton, at first sight, your changes (both this one and the one w.r.t. *USE-REPLACEMENT-CHAR* that you sent later) look reasonable and make a lot of sense. I like the idea of getting rid of two global special variables without sacrificing performance. Would you want to provide a tested patch that also updates the documentation accordingly? Thanks a lot, Edi. From vodonosov at mail.ru Sun Feb 25 21:44:35 2007 From: vodonosov at mail.ru (Anton Vodonosov) Date: Sun, 25 Feb 2007 23:44:35 +0200 Subject: [flexi-streams-devel] refactoring *provide-use-value-restart* In-Reply-To: References: <45E0B8A0.1020601@mail.ru> Message-ID: <45E20343.1000702@mail.ru> Edi Weitz: > Would you want to provide a tested patch that also updates the > documentation accordingly? > Ok, I'll send it soon. Regards, -Anton