Future::isReady() should use memory_order_acquire

XMLWordPrintableJSON

    • Type: Bug
    • Resolution: Fixed
    • Priority: Major - P3
    • 4.1.1
    • Affects Version/s: None
    • Component/s: Internal Code
    • None
    • Fully Compatible
    • ALL
    • Platforms 2018-07-02
    • None
    • None
    • None
    • None
    • None
    • None
    • None

      While the rationale for making it relaxed is probably valid for external callers, an internal caller[1] is already using it incorrectly. Rather than fixing the internal caller, it seems easier and less bug-prone to just make isReady() do an acquire-load.

      [1] That method is only called from its unittest at the moment, so there is no bug in the shipping binaries.

            Assignee:
            Mathias Stearn
            Reporter:
            Mathias Stearn
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved: