jakartaee / expression-language

Jakarta Expression Language
https://eclipse.org/ee4j/el
Other
62 stars 49 forks source link

Custom service loader implementation incomplete #118

Closed markt-asf closed 4 years ago

markt-asf commented 4 years ago

The EL API currently uses a custom service loader implementation. This custom implementation does not skip comment lines so often fails to load valid EL implementations on the class path.

I propose to replace the custom service loader implementation with the use of ServiceLoader.

This issue was identified when working on the TCK updates for the 4.0.0 release. When testing the combination of the Jakarta EL 4.0.0 API and the Tomcat 10 implementation, the implementation failed to load.