Note sub-optimal API design

See #115

Author: petertodd

bitcoin/core/serialize.py

@@ -236,6 +236,11 @@ def stream_deserialize(cls, f):
 
 class VectorSerializer(Serializer):
     """Base class for serializers of object vectors"""
+
+    # FIXME: stream_(de)serialize don't match the signatures of the base class
+    # due to the inner_cls parameter. This probably isn't optimal API design
+    # and should be rethought at some point.
+
     @classmethod
     def stream_serialize(cls, inner_cls, objs, f):
         VarIntSerializer.stream_serialize(len(objs), f)