[Rd] Altrep and translations (was "[R] Description of error is untranslated when ....")
Ricardo Villalba
r|k|v|||@|b@ @end|ng |rom gm@||@com
Tue May 14 16:15:01 CEST 2024
Hello
On the other hand, i find that R_compact_intrange checks whether (r = n2 -
n1 + 1) >= R_XLEN_T_MAX, while seq_colon checks for (r = fabs(n2 - n1)) >=
R_XLEN_T_MAX. And only then n is defined as r + 1 + epsilon. r is not used
again
That is why seq_colon does not reject that size at first and then altrep
does.
Should not line 101 be " if (r + 1 + FLT_EPSILON >= R_XLEN_T_MAX) "
instead of r >= R_XLEN_T_MAX?
El mar, 14 may 2024 a las 4:45, Martin Maechler (<maechler using stat.math.ethz.ch>)
escribió:
> >>>>> Ivan Krylov via R-help
> >>>>> on Tue, 14 May 2024 08:08:58 +0300 writes:
>
> > Dear Ricardo Villalba, Thank you for spotting this corner
> > case!
>
> > В Mon, 13 May 2024 11:37:57 -0300 Ricardo Villalba
> > <rikivillalba using gmail.com> пишет:
>
> >> I track the messages to be coded here:
> >>
> https://github.com/r-devel/r-svn/blob/abe625945c4402cd2bb97b5a64e7469db3e904f0/src/main/altclasses.c#L580
> >> and here
> >>
> https://github.com/r-devel/r-svn/blob/abe625945c4402cd2bb97b5a64e7469db3e904f0/src/main/seq.c#L102
>
> > You may have noticed that one of these places lacks the
> > underscore that R uses as the translation keyword: it's
> > _(string) that looks up the translation of the string.
>
> > Would you like to suggest a patch on R-devel using r-project.org
> > or https://bugs.r-project.org ?
>
> > --
> > Best regards, Ivan
>
> Thank you, but, well... for such small changes -- that would be
> quite a bit of (human, i.e. R-core) overhead, but of course is
> generally a good possibility (but here, wait! .. and read on : )
>
> Actually -- this now gets more "R-devel" - appropriate --
> I notice that there are quite a few error("..." .. )
> and also warning("..." ..) calls in
> src/main/altclasses.c *and* in
> src/main/altrep.c and *none* of them asks for translation;
>
> While this could be for historical reasons (code originally much
> by Gabe Becker from outside R-core *and* at first just in a
> separate branch before being merged in to the main (r-devel)
> branch.
> OTOH: There may be good reasons for translations lookup being
> brittle in case of altrep error messages .. and hence left off
> purposely?
>
> Martin
>
> --
> Martin Maechler
> ETH Zurich and R Core Team
>
>
--
®DV!
[[alternative HTML version deleted]]
More information about the R-devel
mailing list