Make loops less deep
/cc @denittis
Based on !195 (merged).
-
runtime: Factor out a nested loop
Instead of having a function to make a list of Vulkan layers and/or ICDs available, let's have a function to act on a single layer or ICD.
Merge request reports
Activity
The diff for "runtime: Factor out a nested loop" is intimidatingly large, but if you use
git show --ignore-space-change
it becomes easier to see what's happening.The same trick helps for "runtime: Factor out a deeply nested loop", but not as much as I'd like - sorry. Comparing side-by-side might work, or if you're not comfortable with applying this change, I can try to do it some other way by refactoring from the inside out instead of from the outside in.
added 10 commits
-
cd0c7600...64a8d67a - 7 commits from branch
master
- fc314c8c - runtime: Rename some misleading variables
- 4bc0958b - runtime: Rename another misleading variable
- 3b66327b - runtime: Factor out a nested loop
Toggle commit list-
cd0c7600...64a8d67a - 7 commits from branch
mentioned in merge request !197 (merged)
mentioned in commit 8bf7d1d3