Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix variable values thread safety in pure mode #183

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Fix variable values thread safety in pure mode #183

wants to merge 2 commits into from

Commits on Nov 5, 2024

  1. pure: Add test for phax#182 

    This test currently fails because the value of the "var" variable" set
by one thread is overwritten by the other thread.
Theoretically this test would fail or not depending on how the 2 threads
are executed and interweaved. But it seems to always fail, so no further
complication is added for now.
    @bertrand-lorentz
    bertrand-lorentz committed Nov 5, 2024
    Configuration menu
    Copy the full SHA
    a2b30c0 View commit details
    Browse the repository at this point in the history

  2. pure: Store variable values in a ThreadLocal map 

    To ensure that variable values are not overwritten when different XML
are validated on separate threads, store the variable values in aborts
ThreadLocal map.

Fixes issue phax#182.
    @bertrand-lorentz
    bertrand-lorentz committed Nov 5, 2024
    Configuration menu
    Copy the full SHA
    b727406 View commit details
    Browse the repository at this point in the history