Merge pull request #406 from youdie006/symbol-unpacker-only-npe

Guard packerProc against a missing :packer key on JRuby

Author: byroot

ext/java/org/msgpack/jruby/Factory.java

@@ -112,7 +112,7 @@ public IRubyObject registerTypeInternal(ThreadContext ctx, IRubyObject type, IRu
     extensionRegistry.put(extModule, (int) typeId, recursive, packerProc, unpackerProc);
 
     if (extModule == runtime.getSymbol()) {
-      hasSymbolExtType = !packerProc.isNil();
+      hasSymbolExtType = packerProc != null && !packerProc.isNil();
     }
 
     if (options != null) {

spec/factory_spec.rb

@@ -273,6 +273,12 @@ class MyType2 < MyType
       expect(factory.load(factory.dump(:foo))).to be == "foo"
     end
 
+    it 'handles Symbol type registered with only an unpacker' do
+      factory = MessagePack::Factory.new
+      factory.register_type(0x00, Symbol, unpacker: :to_sym.to_proc)
+      expect(factory.load(factory.dump(:foo))).to be == "foo"
+    end
+
     describe "registering multiple ext type for the same class" do
       let(:payload) do
         factory = MessagePack::Factory.new