search

opendatalab-de / geojson-jackson

GeoJson POJOs for Jackson - serialize and deserialize objects with ease
http://blog.opendatalab.de
Apache License 2.0
263 stars 94 forks source link

LngLatAltSerializer#serialize throws useless JsonProcessingException #40

Closed pigelvy closed 7 years ago

pigelvy commented 7 years ago

IntelliJ code inspection warns me that: "There is a more general exception, 'java.io.IOException', in the throws list already"

Declaring "JsonProcessingException" in the throws list is thus useless! This is a more general problem of jackson-databind but I think we can change the method signature without introducing regressions.

What do you think?

pigelvy commented 7 years ago

I also restored the previous code formatting (prior to e8409a9d38019a88cc74bddb6cce85f244728a7a)