Here is the updated webrev:
http://cr.openjdk.java.net/~naoto/8159781/webrev.01/
Naoto
On 6/20/16 8:18 AM, Naoto Sato wrote:
Hi Mandy, thank you for the review.
INCLUDE_LOCALE_FILES is actually a pattern template, and the real filter
patterns are constructed at the runtime with the passed argument, by
replacing "%%" with locale patterns. So I think it is less complex to
handle it as the current way.
I will modify the "regex:" prepending as you suggested.
Naoto
On 6/17/16 3:50 PM, Mandy Chung wrote:
On Jun 17, 2016, at 3:29 PM, Naoto Sato <naoto.s...@oracle.com> wrote:
Decided to fix this separately from the other include locales issues.
Here is the bug and the proposed fix:
https://bugs.openjdk.java.net/browse/JDK-8159781
http://cr.openjdk.java.net/~naoto/8159781/webrev.00/
The change looks okay. This can be converted to use
ResourceFilter::includeFilter(List<String> patterns) rather than
building a comma-separated patterns string. That means META_FILES and
INCLUDE_LOCALE_FILES can be made as List<String>. You have an option
to prepend “regex:” when constructing the patterns to pass to
ResourceFilter. I think that’s a good clean up to do.
Mandy
