Page MenuHomePhabricator

[StringExtras] Rename SubsequentDelim to ListSeparator

Authored by kazu on Wed, Jan 13, 8:39 PM.



This patch renames SubsequentDelim to ListSeparator to clarify the
purpose of the class.

Diff Detail

Event Timeline

kazu created this revision.Wed, Jan 13, 8:39 PM
kazu requested review of this revision.Wed, Jan 13, 8:39 PM
Herald added a project: Restricted Project. · View Herald TranscriptWed, Jan 13, 8:39 PM
dblaikie accepted this revision.Wed, Jan 13, 8:49 PM

Sounds good to me - but give it a day or two. @MaskRay or others might want to weigh in.

This revision is now accepted and ready to land.Wed, Jan 13, 8:49 PM
MaskRay accepted this revision.Wed, Jan 13, 9:01 PM

Thanks. The new name looks better! I mentioned the patch on discord. Hope one day or two is sufficient for interested folks to weigh in.


Not sure whether it should be committed separately:
using LS the third time is useful but untested.

This revision was automatically updated to reflect the committed changes.