thSoft / elysium

LilyPond IDE for Eclipse
http://elysium.thsoft.hu
14 stars 3 forks source link

check for usage of removed guava API #180

Closed nittka closed 6 years ago

nittka commented 6 years ago

175 showed that the development version of guava may be incomatible with the runtime version (due to removed API). We should check our code for usage of API that has been removed "recently"

nittka commented 6 years ago

Using Oxygen and Xtext 2.13 in the target platform, there is one compile error: LilyPondImportUri#toString uses Objects.toStringHelper which has been moved to MoreObjects. I think, we do not call this method in productive code (for logging etc.). My suggestion is to implement the toString method directly, thus being compatible with all guava versions in the range supported by Xtext.

nittka commented 6 years ago

Resolved with merging #174.