diff options
| author | Arseny Kapoulkine <arseny.kapoulkine@gmail.com> | 2017-02-03 20:33:40 -0800 |
|---|---|---|
| committer | Arseny Kapoulkine <arseny.kapoulkine@gmail.com> | 2017-02-03 20:33:40 -0800 |
| commit | bcc7ed57a2b6155c30ec871bb12ffae110fcf558 (patch) | |
| tree | 85d5e50524ce23b8f226ce3c1e8dc84280dba34b /src | |
| parent | 33159924b121f705428b26d1a1649735e1abbf19 (diff) | |
XPath: Simplify evaluation error flow
Instead of having two checks for out-of-memory when exceptions are
enabled, do just one and decide what to do based on whether we can
throw.
Diffstat (limited to 'src')
| -rw-r--r-- | src/pugixml.cpp | 65 |
1 files changed, 45 insertions, 20 deletions
diff --git a/src/pugixml.cpp b/src/pugixml.cpp index 9d1be44..7e6fe64 100644 --- a/src/pugixml.cpp +++ b/src/pugixml.cpp @@ -11823,11 +11823,16 @@ PUGI__NS_BEGIN xpath_string r = impl->root->eval_string(c, sd.stack); - #ifndef PUGIXML_NO_EXCEPTIONS - if (sd.oom) throw std::bad_alloc(); - #endif + if (sd.oom) + { + #ifdef PUGIXML_NO_EXCEPTIONS + return xpath_string(); + #else + throw std::bad_alloc(); + #endif + } - return sd.oom ? xpath_string() : r; + return r; } PUGI__FN impl::xpath_ast_node* evaluate_node_set_prepare(xpath_query_impl* impl) @@ -12463,11 +12468,16 @@ namespace pugi bool r = static_cast<impl::xpath_query_impl*>(_impl)->root->eval_boolean(c, sd.stack); - #ifndef PUGIXML_NO_EXCEPTIONS - if (sd.oom) throw std::bad_alloc(); - #endif + if (sd.oom) + { + #ifdef PUGIXML_NO_EXCEPTIONS + return false; + #else + throw std::bad_alloc(); + #endif + } - return sd.oom ? false : r; + return r; } PUGI__FN double xpath_query::evaluate_number(const xpath_node& n) const @@ -12479,11 +12489,16 @@ namespace pugi double r = static_cast<impl::xpath_query_impl*>(_impl)->root->eval_number(c, sd.stack); - #ifndef PUGIXML_NO_EXCEPTIONS - if (sd.oom) throw std::bad_alloc(); - #endif + if (sd.oom) + { + #ifdef PUGIXML_NO_EXCEPTIONS + return impl::gen_nan(); + #else + throw std::bad_alloc(); + #endif + } - return sd.oom ? impl::gen_nan() : r; + return r; } #ifndef PUGIXML_NO_STL @@ -12527,11 +12542,16 @@ namespace pugi impl::xpath_node_set_raw r = root->eval_node_set(c, sd.stack, impl::nodeset_eval_all); - #ifndef PUGIXML_NO_EXCEPTIONS - if (sd.oom) throw std::bad_alloc(); - #endif + if (sd.oom) + { + #ifdef PUGIXML_NO_EXCEPTIONS + return xpath_node_set(); + #else + throw std::bad_alloc(); + #endif + } - return sd.oom ? xpath_node_set() : xpath_node_set(r.begin(), r.end(), r.type()); + return xpath_node_set(r.begin(), r.end(), r.type()); } PUGI__FN xpath_node xpath_query::evaluate_node(const xpath_node& n) const @@ -12544,11 +12564,16 @@ namespace pugi impl::xpath_node_set_raw r = root->eval_node_set(c, sd.stack, impl::nodeset_eval_first); - #ifndef PUGIXML_NO_EXCEPTIONS - if (sd.oom) throw std::bad_alloc(); - #endif + if (sd.oom) + { + #ifdef PUGIXML_NO_EXCEPTIONS + return xpath_node(); + #else + throw std::bad_alloc(); + #endif + } - return sd.oom ? xpath_node() : r.first(); + return r.first(); } PUGI__FN const xpath_parse_result& xpath_query::result() const |