Follow @Openwall on Twitter for new release announcements and other news
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <CACcSVPHenAyLbCWt+=o2O7rObGYi-minCF0oPNtQOahYB9-HTQ@mail.gmail.com>
Date: Wed, 3 Apr 2019 12:29:23 -0700
From: Dan Gohman <sunfish@...illa.com>
To: musl@...ts.openwall.com
Subject: Re: [PATCH] Fix unintended global symbols in atanl.c

My apologies. Here is the patch as an attachment.

Dan

On Wed, Apr 3, 2019 at 7:16 AM Rich Felker <dalias@...c.org> wrote:

> On Wed, Apr 03, 2019 at 05:56:08AM -0700, Dan Gohman wrote:
> > Mark atanhi, atanlo, and aT in atanl.c as static, as they're not intended
> > to be part of the public API.
> >
> > These are already static in the LDBL_MANT_DIG == 64 code, so this patch
> is
> > just making the LDBL_MANT_DIG == 113 code do the same thing.
> > ---
> >  src/math/atanl.c | 6 +++---
> >  1 file changed, 3 insertions(+), 3 deletions(-)
> >
> > diff --git a/src/math/atanl.c b/src/math/atanl.c
> > index 79a3edb8..c3b0c926 100644
> > --- a/src/math/atanl.c
> > +++ b/src/math/atanl.c
> > @@ -70,21 +70,21 @@ static long double T_odd(long double x)
> >  #elif LDBL_MANT_DIG == 113
> >  #define EXPMAN(u) ((u.i.se & 0x7fff)<<8 | u.i.top>>8)
> >
> > -const long double atanhi[] = {
> > +static const long double atanhi[] = {
> >          4.63647609000806116214256231461214397e-01L,
> >          7.85398163397448309615660845819875699e-01L,
> >          9.82793723247329067985710611014666038e-01L,
> >          1.57079632679489661923132169163975140e+00L,
> >  };
> >
> > -const long double atanlo[] = {
> > +static const long double atanlo[] = {
> >          4.89509642257333492668618435220297706e-36L,
> >          2.16795253253094525619926100651083806e-35L,
> >         -2.31288434538183565909319952098066272e-35L,
> >          4.33590506506189051239852201302167613e-35L,
> >  };
> >
> > -const long double aT[] = {
> > +static const long double aT[] = {
> >          3.33333333333333333333333333333333125e-01L,
> >         -1.99999999999999999999999999999180430e-01L,
> >          1.42857142857142857142857142125269827e-01L,
> > --
> > 2.17.1
>
> Your email system appears to have corrupted the patch; it does not
> apply. Can you please resend as an attachment in the same git
> format-patch form?
>
> Rich
>

Content of type "text/html" skipped

View attachment "0001-Fix-unintended-global-symbols-in-atanl.c.patch" of type "text/x-patch" (1549 bytes)

Powered by blists - more mailing lists

Confused about mailing lists and their use? Read about mailing lists on Wikipedia and check out these guidelines on proper formatting of your messages.