summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorArseny Kapoulkine <arseny.kapoulkine@gmail.com>2014-11-05 08:52:32 +0100
committerArseny Kapoulkine <arseny.kapoulkine@gmail.com>2014-11-05 08:52:32 +0100
commit2cb5d409565d8ee0f2e03312b5699ef454f8d70c (patch)
tree01ecbf7082ec4bd546ac4ff8ff0d4139ace41159
parent3950ee0433aaed3c35eaee7548bae5b00ee22a80 (diff)
Remove redundant branches
These used to result in better codegen for unknown reasons, but this is no longer the case.
-rw-r--r--src/pugixml.cpp6
1 files changed, 0 insertions, 6 deletions
diff --git a/src/pugixml.cpp b/src/pugixml.cpp
index c8de68a..d7d25ba 100644
--- a/src/pugixml.cpp
+++ b/src/pugixml.cpp
@@ -286,8 +286,6 @@ PUGI__NS_BEGIN
{
static xml_memory_page* construct(void* memory)
{
- if (!memory) return 0; //$ redundant, left for performance
-
xml_memory_page* result = static_cast<xml_memory_page*>(memory);
result->allocator = 0;
@@ -2670,15 +2668,11 @@ PUGI__NS_BEGIN
a->name = s; // Save the offset.
PUGI__SCANWHILE_UNROLL(PUGI__IS_CHARTYPE(ss, ct_symbol)); // Scan for a terminator.
- PUGI__CHECK_ERROR(status_bad_attribute, s); //$ redundant, left for performance
-
PUGI__ENDSEG(); // Save char in 'ch', terminate & step over.
- PUGI__CHECK_ERROR(status_bad_attribute, s); //$ redundant, left for performance
if (PUGI__IS_CHARTYPE(ch, ct_space))
{
PUGI__SKIPWS(); // Eat any whitespace.
- PUGI__CHECK_ERROR(status_bad_attribute, s); //$ redundant, left for performance
ch = *s;
++s;