help / color / mirror / Atom feed
From: Blady <>
Subject: Re: [ANN] Release of UXStrings 0.5.0
Date: Sat, 1 Jul 2023 16:41:37 +0200	[thread overview]
Message-ID: <u7pdv2$2v1de$> (raw)
In-Reply-To: <>

Hello Vincent,

Le 29/06/2023 à 10:49, Vincent D. a écrit :
> Hello Pascal,
> Thank you for this contribution. Here are some comments:
> - since UTFString is a class ("a tagged record type"), why don't you create an abstract root "UXString" and then derive specialized object types ? Like UTF_8_XString, UTF_16_XString, ASCII_XString, Win_1252_XString, Latin_XString, etc.

Well, that's a possibility chosen in some other Ada Strings libraries.
I've preferred that the API of legacy Ada "string" types to be closed to 
those of Ada library so that the adaptation would be easy.
These are not intended to be used outside legacy code adaptation.
Note that I've renamed them as character arrays rather than strings in 
order to accentuate the semantic difference.

> - The default format to convert between different encodings should be UTF-8 as it is now ubiquitous.

Conversions are between UXString and encodings, not between encodings.

>> [...] moreover in the case of strings accentuated in French and strings containing emojis the process times are also improved (factor 7 to 8 by compared to UXStrings1
> - I find quite astonishing to have a factor 8 compared to UTF-8 encoding. Do you have an explanation ? This looks like a poor implementation because UTF-8 encoding is fast and allows direct manipulation in most cases. Maybe because random access is treated as sequential access for UTF-8 encoded strings but this again is poor implementation.

You got it: "most cases". Apart from complex implementations, if you 
want to access a specific position you have to parse from the beginning 
of the UTF-8 data as UXStrings1 does.
UXStrings2 always computes if the resulting data are all ASCII, so the 
access is then direct.
UXStrings3 is internally like an Unicode array, so the access is direct.

Best regards, Pascal.

      reply	other threads:[~2023-07-01 14:41 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-05-05  3:36 [ANN] Release of UXStrings 0.5.0 Blady
2023-06-29  8:49 ` Vincent D.
2023-07-01 14:41   ` Blady [this message]
replies disabled

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox