On Wed, 16 Sep 2026 13:11:56 GMT, Timofei Fedotov <[email protected]> wrote:
>> src/java.naming/share/classes/javax/naming/ldap/Rdn.java line 172: >> >>> 170: throw new InvalidNameException( >>> 171: "RDN cannot be empty, got: \"" + rdnString + "\""); >>> 172: } >> >> This is what I meant: >> >> Suggestion: >> >> assert !entries.isEmpty(); >> >> With your fix, the parser is already supposed to throw in that case, so we >> can simply assert here. > > Okay done ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/32648#discussion_r4026450263
