Handling different encodings for repeated fields
joto opened this issue · 1 comments
The protobuf spec says that packed repeated fields must also be parsed if they are not using the packed format and that multiple packed fields are also allowed and should be concatenated. This affects the tags and geometry fields in the features.
The current vtzero code doesn't handle this properly. It is rather unlikely that this will happen in real life, but we have to think about whether we want to allow this. The problem is that this will most likely be more expensive to handle properly. I'll have to think about the details here some more.
As per voice discussion, we'll not handle this case in vtzero to keep performance optimal. This is now documented in https://github.com/mapbox/vtzero/blob/master/doc/advanced.md .