diff options
author | Edward Welbourne <edward.welbourne@qt.io> | 2017-06-08 12:19:23 +0200 |
---|---|---|
committer | Edward Welbourne <edward.welbourne@qt.io> | 2017-06-09 08:24:33 +0000 |
commit | e93af7dafd4d45cfb48efb766080964f666b4f87 (patch) | |
tree | d24b0bcf150c94cb6abf46c9db0d5baca7edea3a /config.tests/unix | |
parent | ad1cec5a5eeebbd7b2c657e4e2aff913815344e9 (diff) |
Turn a next-node function into an iterator over nodes
This simplifies a duplicated iteration pattern in the calling code.
It also frees the first-node function to raise an Error (which the
iterator now catches) if it finds no node, where it used to return
False (instead of a node, so other code using it would raise confusing
errors from trying to use False as a node, where now it'll get a clear
Error about a missing node). There were also no callers passing an
empty name, so the test for matching nodes (here moved to its own
short function) didn't need to handle that as a special case.
Change-Id: Ife6cad8943cf5dc2c6ed68429d4a217cb9bea446
Reviewed-by: Lars Knoll <lars.knoll@qt.io>
Diffstat (limited to 'config.tests/unix')
0 files changed, 0 insertions, 0 deletions