[Commits] [wesnoth/wesnoth] 5cf272: openmp: don't parallelize anim refresh() calls in ...

GitHub noreply at github.com
Fri Jan 2 09:16:05 UTC 2015


  Branch: refs/heads/master
  Home:   https://github.com/wesnoth/wesnoth
  Commit: 5cf272f7dc336d2449fb64d388a4d29d9b2f4c37
      https://github.com/wesnoth/wesnoth/commit/5cf272f7dc336d2449fb64d388a4d29d9b2f4c37
  Author: Chris Beck <render787 at gmail.com>
  Date:   2015-01-02 (Fri, 02 Jan 2015)

  Changed paths:
    M src/display.cpp

  Log Message:
  -----------
  openmp: don't parallelize anim refresh() calls in display.cpp

This commit fixes up an earlier commit
34a745469667ef55ef5afd7ced080908be63c1fb

In that commit, unit animation function refresh() was paralellized
as well as the invalidate() function. Vultraz has reported
assertion failures in openmp and this seems like a probable cause.
See irclogs from Jan 02 2015 for more discussion.


  Commit: 7f4a6b80fbaad468461a84bd5df2fc4daada6910
      https://github.com/wesnoth/wesnoth/commit/7f4a6b80fbaad468461a84bd5df2fc4daada6910
  Author: Chris Beck <render787 at gmail.com>
  Date:   2015-01-02 (Fri, 02 Jan 2015)

  Changed paths:
    M src/display.cpp

  Log Message:
  -----------
  display: add a comment about order of unit drawing

in invalidate_animations(), add a comment about the significance
of the order of certain steps


  Commit: 4077896b55f17f52d779da359eee6d5fcbbcb437
      https://github.com/wesnoth/wesnoth/commit/4077896b55f17f52d779da359eee6d5fcbbcb437
  Author: Chris Beck <beck.ct at gmail.com>
  Date:   2015-01-02 (Fri, 02 Jan 2015)

  Changed paths:
    M src/display.cpp

  Log Message:
  -----------
  Merge pull request #357 from cbeck88/fix_openmp

Fix openmp


Compare: https://github.com/wesnoth/wesnoth/compare/7625ece98f6a...4077896b55f1


More information about the Commits mailing list