You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@opennlp.apache.org by "Martin Wiesner (Jira)" <ji...@apache.org> on 2026/08/25 07:49:00 UTC
[jira] [Assigned] (OPENNLP-1916) Improve Normalizer documentation and fail loud on chain
[ https://issues.apache.org/jira/browse/OPENNLP-1916?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Martin Wiesner reassigned OPENNLP-1916:
---------------------------------------
Assignee: Kristian Rickert
> Improve Normalizer documentation and fail loud on chain
> -------------------------------------------------------
>
> Key: OPENNLP-1916
> URL: https://issues.apache.org/jira/browse/OPENNLP-1916
> Project: OpenNLP
> Issue Type: Improvement
> Affects Versions: 3.0.0-M5
> Reporter: Kristian Rickert
> Assignee: Kristian Rickert
> Priority: Minor
> Fix For: 3.0.0-M6
>
> Time Spent: 10m
> Remaining Estimate: 0h
>
> 1. Minor: Replace the rung metaphor with "normalizer" wording.
> 2. Minor:
> Validate and copy the aggregate normalizer chain at construction: AggregateCharSequenceNormalizer stored the caller's varargs array as-is (null array or null element caused NPE mid-normalize, which cascaded into a changed aggregate).
>
> I changed it to fail loudly.
>
> Now throws IllegalArgumentException on null, defensive copy, shadow gone, constructor javadoc added, plus a new 4-test AggregateCharSequenceNormalizerTest covering order, both null rejections, and array-mutation isolation.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)