Skip to content

M
mpd
Commit 9f21eee2 authored by Max Kellermann's avatar Max Kellermann
Browse files
Options

ApeTag: simplify the "recognized" flag

parent a40246d3
Hide whitespace changes
Inline Side-by-side
Showing with 1 addition and 4 deletions
src/tag/ApeTag.cxx
View file @ 9f21eee2
...@@ -63,7 +63,6 @@ tag_ape_import_item(unsigned long flags, ...@@ -63,7 +63,6 @@ tag_ape_import_item(unsigned long flags,
if (type == TAG_NUM_OF_ITEM_TYPES) if (type == TAG_NUM_OF_ITEM_TYPES)
return false; return false;
bool recognized = false;
const char *end = value + value_length; const char *end = value + value_length;
while (true) { while (true) {
/* multiple values are separated by null bytes */ /* multiple values are separated by null bytes */
...@@ -72,7 +71,6 @@ tag_ape_import_item(unsigned long flags, ...@@ -72,7 +71,6 @@ tag_ape_import_item(unsigned long flags,
if (n > value) { if (n > value) {
tag_handler_invoke_tag(handler, handler_ctx, tag_handler_invoke_tag(handler, handler_ctx,
type, value); type, value);
recognized = true;
} }
value = n + 1; value = n + 1;
...@@ -80,12 +78,11 @@ tag_ape_import_item(unsigned long flags, ...@@ -80,12 +78,11 @@ tag_ape_import_item(unsigned long flags,
const std::string value2(value, end); const std::string value2(value, end);
tag_handler_invoke_tag(handler, handler_ctx, tag_handler_invoke_tag(handler, handler_ctx,
type, value2.c_str()); type, value2.c_str());
recognized = true;
break; break;
} }
} }
return recognized; return true;
} }
bool bool
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment