From a3cec3b000a9bb6f08a19e38dfd4a3d858d40374 Mon Sep 17 00:00:00 2001
From: Moritz Bunkus <moritz@bunkus.org>
Date: Sat, 1 Aug 2020 18:40:29 +0200
Subject: [PATCH] mkvmerge: fix decision whether or not to write
 BlockAdditionMapping

BlockAddIDType is mandatory, all other elements optional. Therefore
require the ID type to be set but don't care about the others.

Part of the implementation of #2887.
---
 src/merge/block_addition_mapping.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/merge/block_addition_mapping.h b/src/merge/block_addition_mapping.h
index 278d5fc24..f27acd4af 100644
--- a/src/merge/block_addition_mapping.h
+++ b/src/merge/block_addition_mapping.h
@@ -19,6 +19,6 @@ struct block_addition_mapping_t {
   memory_cptr id_extra_data;
 
   bool is_valid() const {
-    return id_type || id_value || (id_extra_data && id_extra_data->get_size());
+    return !!id_type;
   }
 };