[12gevmoller_sim] [JeffersonLab/remoll] 17178a: Cosmetic change only

Wouter Deconinck noreply at github.com
Tue Apr 9 12:06:17 EDT 2019


  Branch: refs/heads/bugfix-issue-232-threadsafe-magnetic-field
  Home:   https://gcc01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FJeffersonLab%2Fremoll&data=02%7C01%7C12gevmoller_sim%40jlab.org%7Cacd61aaf0ca34e4f0a3f08d6bd054d64%7Cb4d7ee1f4fb34f0690372b5b522042ab%7C1%7C1%7C636904227803596734&sdata=%2FUjXOkuJOJ1cUVNGRPkTodzt9t4S%2Bc0P0IGlGBtwvYg%3D&reserved=0
  Commit: 17178a102f1ca7260eca968d873abf666122f703
      https://gcc01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FJeffersonLab%2Fremoll%2Fcommit%2F17178a102f1ca7260eca968d873abf666122f703&data=02%7C01%7C12gevmoller_sim%40jlab.org%7Cacd61aaf0ca34e4f0a3f08d6bd054d64%7Cb4d7ee1f4fb34f0690372b5b522042ab%7C1%7C1%7C636904227803596734&sdata=aCMBIYaXeYLv4WlZnQzOWFC9uBJSdRqrmsSDFNupnHk%3D&reserved=0
  Author: Wouter Deconinck <wdconinc at gmail.com>
  Date:   2019-04-09 (Tue, 09 Apr 2019)

  Changed paths:
    M src/remollMagneticField.cc

  Log Message:
  -----------
  Cosmetic change only


  Commit: 3f90c66b32893efda4fd8d8d34ca1776bc8c9d4a
      https://gcc01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FJeffersonLab%2Fremoll%2Fcommit%2F3f90c66b32893efda4fd8d8d34ca1776bc8c9d4a&data=02%7C01%7C12gevmoller_sim%40jlab.org%7Cacd61aaf0ca34e4f0a3f08d6bd054d64%7Cb4d7ee1f4fb34f0690372b5b522042ab%7C1%7C1%7C636904227803596734&sdata=2KLCRpVcBl6zb0ezv8VpQne%2FuJRaZy45RYU2etjab7o%3D&reserved=0
  Author: Wouter Deconinck <wdconinc at gmail.com>
  Date:   2019-04-09 (Tue, 09 Apr 2019)

  Changed paths:
    M include/remollGlobalField.hh
    M src/remollGlobalField.cc

  Log Message:
  -----------
  Make remollMagneticFields static in remollGlobalField

This ensures that only a single set of fields takes up memory. Also made
sure that the fields are only read in once. Added mutexes to avoid
threading errors in issue #232 (which should now be resolved). Tested by
running successfully with 64 threads (on my laptop, not very
repeatable).


Compare: https://gcc01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FJeffersonLab%2Fremoll%2Fcompare%2F17178a102f1c%255E...3f90c66b3289&data=02%7C01%7C12gevmoller_sim%40jlab.org%7Cacd61aaf0ca34e4f0a3f08d6bd054d64%7Cb4d7ee1f4fb34f0690372b5b522042ab%7C1%7C1%7C636904227803596734&sdata=cwcK7LculhORCVG76K2hjh%2B75WhEvyPV5jd5z%2B7DiDs%3D&reserved=0


More information about the 12gevmoller_sim mailing list