search for: luyssen

On 01/06/2017 01:53 PM, Hugo Beauzée-Luyssen wrote: > Hi, > > This patchet allows a few targets to be disabled when unrequired. > The rational is coming from VLC's contrib buildsystem, so far we use make -C to select only some subparts of the available targets. > It would be easier and cleaner to use autoconf to do so IMHO...

..._LTLIBRARIES but failed miserably at doing so :( I'm thinking it would be way easier to achieve by moving to a non-recursive automake configuration, but that's kind of an intrusive change; that being said, I'm willing to do it. Please let me know what you think! Regards, Hugo Beauzée-Luyssen (5): configure.ac: Allow the programs to be disabled configure.ac: Allow bench to be disabled configure.ac: Don't build any tests when they are explicitely disabled configure.ac: Allow examples to be disabled win_utf8_io: Avoid forbidden functions when building for WinRT/UWP configu...

...and _M_X64, as Martin said. I sent a patch to this ML that fixed these a few months ago but it was forgotten/ignored. Hopefully the issue gets addressed this time. Cameron On Nov 6, 2014 9:53 AM, "Martin Storsj?" <martin at martin.st> wrote: > On Thu, 6 Nov 2014, Hugo Beauz?e-Luyssen wrote: > > --- >> celt/float_cast.h | 4 ++-- >> 1 file changed, 2 insertions(+), 2 deletions(-) >> >> diff --git a/celt/float_cast.h b/celt/float_cast.h >> index ede6574..4892e2c 100644 >> --- a/celt/float_cast.h >> +++ b/celt/float_cast.h >> @...

...person managing the list at opus-owner at xiph.org When replying, please edit your Subject line so it is more specific than "Re: Contents of opus digest..." Today's Topics: 1. Re: opus Digest, Vol 70, Issue 1 (Heng Lou) 2. [PATCH] float_cast: Fix MSVC ARM build (Hugo Beauz?e-Luyssen) 3. Re: [PATCH] float_cast: Fix MSVC ARM build (Martin Storsj?) ---------------------------------------------------------------------- Message: 1 Date: Wed, 5 Nov 2014 20:19:04 +0000 From: Heng Lou <Heng_Lou at starkey.com> Subject: Re: [opus] opus Digest, Vol 70, Issue 1 To: "opus...

Hi, The attached patch fixes build on Windows/ARM platform, as it doesn't have inline assembly. Regards, -- Hugo Beauz?e-Luyssen hugo at beauzee.fr -------------- next part -------------- A non-text attachment was scrubbed... Name: fix_msvc_build.patch Type: text/x-patch Size: 489 bytes Desc: not available Url : http://lists.xiph.org/pipermail/vorbis-dev/attachments/20150227/9acf1cbc/attachment.bin

On Thu, 6 Nov 2014, Jean-Marc Valin wrote: > Is there any consensus on what's the correct fix here? Cameron's fix looks correct to me (I didn't read the code closely enough in my first comment). // Martin

On Mon, 10 Nov 2014, Hugo Beauzee-Luyssen wrote: > On 11/06/2014 10:55 PM, Martin Storsj? wrote: >> On Thu, 6 Nov 2014, Jean-Marc Valin wrote: >> >>> Is there any consensus on what's the correct fix here? >> >> Cameron's fix looks correct to me (I didn't read the code closely enough >>...

--- celt/float_cast.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/celt/float_cast.h b/celt/float_cast.h index ede6574..4892e2c 100644 --- a/celt/float_cast.h +++ b/celt/float_cast.h @@ -90,14 +90,14 @@ #include <math.h> #define float2int(x) lrint(x) -#elif (defined(_MSC_VER) && _MSC_VER >= 1400) && (defined (WIN64) || defined (_WIN64))

...s.xiph.org/pipermail/opus/attachments/20140406/e19d33cd/attachment.obj > > Best, > Tristan > > > > > Cameron > > > > On Nov 6, 2014 9:53 AM, "Martin Storsj?" <martin at martin.st> wrote: > >> > >> On Thu, 6 Nov 2014, Hugo Beauz?e-Luyssen wrote: > >> > >>> --- > >>> celt/float_cast.h | 4 ++-- > >>> 1 file changed, 2 insertions(+), 2 deletions(-) > >>> > >>> diff --git a/celt/float_cast.h b/celt/float_cast.h > >>> index ede6574..4892e2c 100644 > >&...

Hugo Beauzée-Luyssen <hugo at beauzee.fr> wrote: > --- > src/share/win_utf8_io/win_utf8_io.c | 9 +++++++++ > 1 file changed, 9 insertions(+) > > diff --git a/src/share/win_utf8_io/win_utf8_io.c > b/src/share/win_utf8_io/win_utf8_io.c > index c61d27f3..1437b41e 100644 > --- a/src/share...

Hugo Beauzée-Luyssen wrote: >> configure.ac: Allow the programs to be disabled >> configure.ac: Allow bench to be disabled >> configure.ac: Don't build any tests when they are explicitely disabled >> configure.ac: Allow examples to be disabled >> win_utf8_io: Avoid forbidden...

On Thu, 6 Nov 2014, Hugo Beauz?e-Luyssen wrote: > --- > celt/float_cast.h | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/celt/float_cast.h b/celt/float_cast.h > index ede6574..4892e2c 100644 > --- a/celt/float_cast.h > +++ b/celt/float_cast.h > @@ -90,14 +90,14 @@ > #include <...

...ssed this time. This one, I gather? http://lists.xiph.org/pipermail/opus/attachments/20140406/e19d33cd/attachment.obj Best, Tristan > > Cameron > > On Nov 6, 2014 9:53 AM, "Martin Storsj?" <martin at martin.st> wrote: >> >> On Thu, 6 Nov 2014, Hugo Beauz?e-Luyssen wrote: >> >>> --- >>> celt/float_cast.h | 4 ++-- >>> 1 file changed, 2 insertions(+), 2 deletions(-) >>> >>> diff --git a/celt/float_cast.h b/celt/float_cast.h >>> index ede6574..4892e2c 100644 >>> --- a/celt/float_cast.h >>...

...gt; Best, > Tristan > > > > > Cameron > > > > On Nov 6, 2014 9:53 AM, "Martin Storsj?" <martin at martin.st > <mailto:martin at martin.st>> wrote: > >> > >> On Thu, 6 Nov 2014, Hugo Beauz?e-Luyssen wrote: > >> > >>> --- > >>> celt/float_cast.h | 4 ++-- > >>> 1 file changed, 2 insertions(+), 2 deletions(-) > >>> > >>> diff --git a/celt/float_cast.h b/celt/float_cast.h > >>> index ede657...

...rtin > _______________________________________________ > opus mailing list > opus at xiph.org > http://lists.xiph.org/mailman/listinfo/opus > That looks good to me as well, I'm happy to drop the original patch in favor of this one! Also, thanks for the review! -- Hugo Beauz?e-Luyssen hugo at beauzee.fr +33-7-83-33-84-08 -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: OpenPGP digital signature Url : http://lists.xiph.org/pipermail/opus/attachments/20141110/c0688c18/attachment...