diff options
author | Yingchi Long <i@lyc.dev> | 2023-07-03 16:02:54 +0800 |
---|---|---|
committer | Yingchi Long <i@lyc.dev> | 2023-07-03 16:05:43 +0800 |
commit | 3c90340fe64fb18e66f02a648ebce8ad629482a5 (patch) | |
tree | f843f180f20cdb60651c5a6dad1f3c26c77080af /src/libexpr | |
parent | 7b39a388b382e7912de3c5951faad42fe2d72f48 (diff) |
libexpr: use `thread_local` to make the parser thread-safe
If we call `adjustLoc`, the global variable `prev_yylloc` is shared
between threads and racy.
Currently, nix itself does not concurrently parsing files, but this is
helpful for libexpr users. (The parser is thread-safe except this.)
Diffstat (limited to 'src/libexpr')
-rw-r--r-- | src/libexpr/lexer.l | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/libexpr/lexer.l b/src/libexpr/lexer.l index 462b3b602..a3a8608d9 100644 --- a/src/libexpr/lexer.l +++ b/src/libexpr/lexer.l @@ -36,7 +36,7 @@ static inline PosIdx makeCurPos(const YYLTYPE & loc, ParseData * data) #define CUR_POS makeCurPos(*yylloc, data) // backup to recover from yyless(0) -YYLTYPE prev_yylloc; +thread_local YYLTYPE prev_yylloc; static void initLoc(YYLTYPE * loc) { |