RFR: Create forges from service providers

Erik Helin ehelin at openjdk.org
Thu Oct 24 09:08:03 UTC 2019


On Thu, 24 Oct 2019 08:51:32 GMT, Robin Westberg <rwestberg at openjdk.org> wrote:

> Hi all,
> 
> Please review this refactoring that allows forges to be created from service providers.
> 
> Best regards,
> Robin
> 
> ----------------
> 
> Commits:
>  - 063bcf6b: Create Forges from service providers
> 
> Changes: https://git.openjdk.java.net/skara/pull/223/files
>  Webrev: https://webrevs.openjdk.java.net/skara/223/webrev.00
>   Stats: 2799 lines in 33 files changed: 1446 ins; 1294 del; 59 mod
>   Patch: https://git.openjdk.java.net/skara/pull/223.diff
>   Fetch: git fetch https://git.openjdk.java.net/skara pull/223/head:pull/223

forge/src/main/java/org/openjdk/skara/forge/Forge.java line 60:

> 59:         return Optional.empty();
> 60:     }
> 61: 

Maybe catch `IllegalArgumentException` instead? A bit more meaning than just `RuntimeException` :)

PR: https://git.openjdk.java.net/skara/pull/223


More information about the skara-dev mailing list