Move some MIDI mapper protobuf reflection code into a shared helper.

diff --git a/nageru/midi_mapper.cpp b/nageru/midi_mapper.cpp
index 30e934b856396f3e450274912c835dfe52f1436c..59637b1d24107473893f8487227c270942a19058 100644 (file)
--- a/nageru/midi_mapper.cpp
+++ b/nageru/midi_mapper.cpp
@@ -20,6 +20,7 @@
 
 #include "audio_mixer.h"
 #include "midi_mapping.pb.h"
+#include "shared/midi_mapper_util.h"
 #include "shared/text_proto.h"
 
 using namespace google::protobuf;
@@ -209,15 +210,7 @@ void MIDIMapper::match_controller(int controller, int field_number, int bank_fie
 
        for (size_t bus_idx = 0; bus_idx < size_t(mapping_proto->bus_mapping_size()); ++bus_idx) {
                const MIDIMappingBusProto &bus_mapping = mapping_proto->bus_mapping(bus_idx);
-
-               const FieldDescriptor *descriptor = bus_mapping.GetDescriptor()->FindFieldByNumber(field_number);
-               const Reflection *bus_reflection = bus_mapping.GetReflection();
-               if (!bus_reflection->HasField(bus_mapping, descriptor)) {
-                       continue;
-               }
-               const MIDIControllerProto &controller_proto =
-                       static_cast<const MIDIControllerProto &>(bus_reflection->GetMessage(bus_mapping, descriptor));
-               if (controller_proto.controller_number() == controller) {
+               if (match_controller_helper(bus_mapping, field_number, controller)) {
                        func(bus_idx, value);
                }
        }
@@ -231,15 +224,7 @@ void MIDIMapper::match_button(int note, int field_number, int bank_field_number,
 
        for (size_t bus_idx = 0; bus_idx < size_t(mapping_proto->bus_mapping_size()); ++bus_idx) {
                const MIDIMappingBusProto &bus_mapping = mapping_proto->bus_mapping(bus_idx);
-
-               const FieldDescriptor *descriptor = bus_mapping.GetDescriptor()->FindFieldByNumber(field_number);
-               const Reflection *bus_reflection = bus_mapping.GetReflection();
-               if (!bus_reflection->HasField(bus_mapping, descriptor)) {
-                       continue;
-               }
-               const MIDIButtonProto &button_proto =
-                       static_cast<const MIDIButtonProto &>(bus_reflection->GetMessage(bus_mapping, descriptor));
-               if (button_proto.note_number() == note) {
+               if (match_button_helper(bus_mapping, field_number, note)) {
                        func(bus_idx);
                }
        }
@@ -259,10 +244,7 @@ bool MIDIMapper::has_active_controller(unsigned bus_idx, int field_number, int b
 
 bool MIDIMapper::bank_mismatch(int bank_field_number)
 {
-       const FieldDescriptor *bank_descriptor = mapping_proto->GetDescriptor()->FindFieldByNumber(bank_field_number);
-       const Reflection *reflection = mapping_proto->GetReflection();
-       return (reflection->HasField(*mapping_proto, bank_descriptor) &&
-               reflection->GetInt32(*mapping_proto, bank_descriptor) != current_controller_bank);
+       return !match_bank_helper(*mapping_proto, bank_field_number, current_controller_bank);
 }
 
 void MIDIMapper::refresh_highlights()

diff --git a/shared/midi_mapper_util.h b/shared/midi_mapper_util.h
new file mode 100644 (file)
index 0000000..bc9b1bf
--- /dev/null
+++ b/shared/midi_mapper_util.h
@@ -0,0 +1,49 @@
+#ifndef _MIDI_MAPPER_UTIL_H
+#define _MIDI_MAPPER_UTIL_H 1
+
+#include "midi_mapping.pb.h"
+
+#include <google/protobuf/descriptor.h>
+
+template <class Proto>
+inline bool match_controller_helper(const Proto &msg, int field_number, int controller)
+{
+       using namespace google::protobuf;
+       const FieldDescriptor *descriptor = msg.GetDescriptor()->FindFieldByNumber(field_number);
+       const Reflection *reflection = msg.GetReflection();
+       if (!reflection->HasField(msg, descriptor)) {
+               return false;
+       }
+       const MIDIControllerProto &controller_proto =
+               static_cast<const MIDIControllerProto &>(reflection->GetMessage(msg, descriptor));
+       return (controller_proto.controller_number() == controller);
+}
+
+template <class Proto>
+inline bool match_button_helper(const Proto &msg, int field_number, int note)
+{
+       using namespace google::protobuf;
+       const FieldDescriptor *descriptor = msg.GetDescriptor()->FindFieldByNumber(field_number);
+       const Reflection *reflection = msg.GetReflection();
+       if (!reflection->HasField(msg, descriptor)) {
+               return false;
+       }
+       const MIDIButtonProto &button_proto =
+               static_cast<const MIDIButtonProto &>(reflection->GetMessage(msg, descriptor));
+       return (button_proto.note_number() == note);
+}
+
+template <class Proto>
+inline bool match_bank_helper(const Proto &msg, int bank_field_number, int bank)
+{
+       using namespace google::protobuf;
+       const FieldDescriptor *bank_descriptor = msg.GetDescriptor()->FindFieldByNumber(bank_field_number);
+       const Reflection *reflection = msg.GetReflection();
+       if (!reflection->HasField(msg, bank_descriptor)) {
+               // No bank set => in all banks.
+               return true;
+       }
+       return reflection->GetInt32(msg, bank_descriptor) == bank;
+}
+
+#endif  // !defined(_MIDI_MAPPER_UTIL_H)