Avoid creating unnessary wrapper for empty map in DefaultPathSegment

Comment From: sbrannen

Thanks for the PR!

It turns out that your proposal introduces a bug, which in turn made me notice an existing bug.

Namely, any map returned by PathSegment#parameters() must be immutable, and this includes the shared empty map (EMPTY_PARAMS).

Comment From: sbrannen

Superseded by #27064