Print

Print


  Branch: refs/heads/master
  Home:   https://github.com/xrootd/xrootd
  Commit: 04f312b8d2001c93d1f513dccb34f307840595c1
      https://github.com/xrootd/xrootd/commit/04f312b8d2001c93d1f513dccb34f307840595c1
  Author: Brian Bockelman <[log in to unmask]>
  Date:   2014-11-18 (Tue, 18 Nov 2014)

  Changed paths:
    M src/XrdCl/XrdClLog.hh

  Log Message:
  -----------
  Use C++11 atomics to avoid undefined behavior.

C++11 does not guarantee that loads / stores are atomic for any
variable.  Using C++11, we force atomicity for the log level - but do not
provide any guarantee about ordering or visibility.

Without this, setting and getting the log level results in undefined
behavior.  Undefined behavior results in illegal C++11.

On any known processor and platform, this should be a no-op; however, the
code is now conforming C++11.


  Commit: 695fcc16d468358dfc318f8e58a46da0754fa6df
      https://github.com/xrootd/xrootd/commit/695fcc16d468358dfc318f8e58a46da0754fa6df
  Author: Lukasz Janyst <[log in to unmask]>
  Date:   2014-11-18 (Tue, 18 Nov 2014)

  Changed paths:
    M src/XrdCl/XrdClLog.hh

  Log Message:
  -----------
  XrdCl: Refactor the atomics patch slightly


Compare: https://github.com/xrootd/xrootd/compare/32895156110b...695fcc16d468

########################################################################
Use REPLY-ALL to reply to list

To unsubscribe from the XROOTD-DEV list, click the following link:
https://listserv.slac.stanford.edu/cgi-bin/wa?SUBED1=XROOTD-DEV&A=1