Re: [LyX/master] Replace static with thread_local when used for caching

2016-08-22 Thread Stephan Witt
Am 05.08.2016 um 17:07 schrieb Guillaume Munch : > > Le 05/08/2016 à 08:39, Stephan Witt a écrit : >> Am 04.08.2016 um 14:55 schrieb Guillaume Munch : >>> >>> Le 04/08/2016 à 08:48, Stephan Witt a écrit : On my Mac with clang I cannot compile after this commit :( src/fronten

Re: [LyX/master] Replace static with thread_local when used for caching

2016-08-05 Thread Guillaume Munch
Le 05/08/2016 à 19:25, Richard Heck a écrit : I've suggested it before, but: Why can't we just test for this in configure? The reason is that testing for its availability is not sufficient. Indeed, one also has to have a ready replacement that does something similar enough. Otherwise, every ti

Re: [LyX/master] Replace static with thread_local when used for caching

2016-08-05 Thread Richard Heck
On 08/05/2016 11:07 AM, Guillaume Munch wrote: > Le 05/08/2016 à 08:39, Stephan Witt a écrit : >> Am 04.08.2016 um 14:55 schrieb Guillaume Munch : >>> >>> Le 04/08/2016 à 08:48, Stephan Witt a écrit : On my Mac with clang I cannot compile after this commit :( src/frontends/qt4/G

Re: [LyX/master] Replace static with thread_local when used for caching

2016-08-05 Thread Guillaume Munch
Le 05/08/2016 à 08:39, Stephan Witt a écrit : Am 04.08.2016 um 14:55 schrieb Guillaume Munch : Le 04/08/2016 à 08:48, Stephan Witt a écrit : On my Mac with clang I cannot compile after this commit :( src/frontends/qt4/GuiFontLoader.cpp:380:2: error: thread-local storage is not supported for

Re: [LyX/master] Replace static with thread_local when used for caching

2016-08-05 Thread Stephan Witt
Am 04.08.2016 um 14:55 schrieb Guillaume Munch : > > Le 04/08/2016 à 08:48, Stephan Witt a écrit : >> >> On my Mac with clang I cannot compile after this commit :( >> >> src/frontends/qt4/GuiFontLoader.cpp:380:2: error: thread-local storage is >> not supported for the current target >>t

Re: [LyX/master] Replace static with thread_local when used for caching

2016-08-04 Thread Guillaume Munch
Le 04/08/2016 à 08:48, Stephan Witt a écrit : On my Mac with clang I cannot compile after this commit :( src/frontends/qt4/GuiFontLoader.cpp:380:2: error: thread-local storage is not supported for the current target thread_local vector cache_set(NUM_FAMILIES, false); src/frontends/qt4/

Re: [LyX/master] Replace static with thread_local when used for caching

2016-08-04 Thread Stephan Witt
Am 02.08.2016 um 02:14 schrieb Guillaume Munch : > > Le 02/08/2016 à 00:18, Richard Heck a écrit : >> On 08/01/2016 06:49 PM, Guillaume Munch wrote: >>> +#if defined(__GNUC__) && (__GNUC__ == 4) && (__GNUC_MINOR__ == 6) >> >> I meant to ask before: Even if we're requiring gcc >= 4.6 (right?), is

Re: [LyX/master] Replace static with thread_local when used for caching

2016-08-01 Thread Guillaume Munch
Le 02/08/2016 à 00:18, Richard Heck a écrit : On 08/01/2016 06:49 PM, Guillaume Munch wrote: +#if defined(__GNUC__) && (__GNUC__ == 4) && (__GNUC_MINOR__ == 6) I meant to ask before: Even if we're requiring gcc >= 4.6 (right?), is it worth changing this to: (__GNUC_MINOR__ <= 6) just in ca

Re: [LyX/master] Replace static with thread_local when used for caching

2016-08-01 Thread Richard Heck
On 08/01/2016 06:49 PM, Guillaume Munch wrote: > diff --git a/src/frontends/qt4/GuiFontLoader.cpp > b/src/frontends/qt4/GuiFontLoader.cpp > index cc092f5..2dacc42 100644 > --- a/src/frontends/qt4/GuiFontLoader.cpp > +++ b/src/frontends/qt4/GuiFontLoader.cpp > @@ -373,9 +373,13 @@ GuiFontInfo::GuiF