CartoDB / mobile-carto-libs

Internal dependencies for CARTO Mobile SDK
BSD 3-Clause "New" or "Revised" License
4 stars 5 forks source link

color parsing error breaks all tile #12

Closed farfromrefug closed 2 years ago

farfromrefug commented 2 years ago

I have a vector tile with a wrongly defined attribute which i use for color. So white-red is passed as a color. The color parser correctly reports the issue with:

MBVectorTileDecoder::decodeTile: Exception while decoding: Could not parse to the end of color, error at position 5: white-red

The thing is that it throws and then breaks the whole tile. So i end up with a missing tile (you can see the limit of the tile in the screenshot). The feature itself should not be drawn, however it should not prevent the whole tile from being drawn, right?

Screenshot 2021-07-15 at 16 09 45
mtehver commented 2 years ago

Right, this failure should not propagate to tile level, dropping related features should be enough. I will fix this.

farfromrefug commented 2 years ago

I think it is fixed now