Apply skip_symbol? check to macro definitions for consistency with variables.

Macro constants and variable constants both render the same "constant"
template, but macros lacked the skip check that variables have via
visit_variable. cursor.spelling works correctly for macro cursors so
skip_symbol? applies cleanly.

Author: cfis

lib/ruby-bindgen/generators/rice/rice.rb

@@ -1729,6 +1729,7 @@ def visit_macro_definition(cursor)
         return unless tokens.size == 2
         return unless tokens.tokens[0].kind == :identifier
         return unless tokens.tokens[1].kind == :literal
+        return if skip_symbol?(cursor)
 
         self.render_cursor(cursor, "constant",
                            :name => tokens.tokens[0].spelling.upcase_first,

test/bindings/cpp/version_guards-rb.cpp

@@ -9,6 +9,8 @@ using namespace Rice;
 
 void Init_VersionGuards()
 {
+  Class(rb_cObject).define_constant("INCLUDED_MACRO", INCLUDED_MACRO);
+
   Module rb_mGuards = define_module("Guards");
 
   Rice::Data_Type<Guards::MyClass> rb_cGuardsMyClass = define_class_under<Guards::MyClass>(rb_mGuards, "MyClass")

test/headers/cpp/version_guards.hpp

@@ -1,5 +1,9 @@
 #pragma once
 
+// Macro constants — SKIPPED_MACRO should be filtered via symbols
+#define INCLUDED_MACRO 100
+#define SKIPPED_MACRO 999
+
 namespace Guards
 {
   // Class with methods, constructors, and anonymous enum constants

test/rice_test.rb

@@ -107,7 +107,8 @@ def test_mappings
   def test_version_guards
     run_rice_test("version_guards.hpp",
                   version_check: "TEST_VERSION",
-                  symbols: { versions: {
+                  symbols: { skip: ["SKIPPED_MACRO"],
+                             versions: {
                     20000 => ["Guards::MyClass::newMethod",
                               "Guards::MyClass::NEW_CONST",
                               "Guards::MyClass::overloaded(int, bool)",