Don't parse word time tags if multiple line times are present

[Maxr1998]

Word time tags are absolute timestamps in the lyrics, so having multiple line times would generally conflict with them. Thus, they should be just ignored instead, since the format is wrong.
The same applies if there is no line time tag, in that case, it wouldn't be known when the first word starts.
Both are inconsidered incorrect LRC and thus only parsed on a best-effort basis.

[andy840119]

I know lrc support multiple line times with one lyric line, but i did not remember the time tag will be dropped.
Is that a lrc spec?
Or this change is made because you feels reasonable?

[Maxr1998]

Since there's no official spec, it's hard to quote one, but this new behavior is reasonable because of the mentioned conflict between line time and absolute word times. I've alwo never seen a mix of multiple line times and word times being used in the wild.

It's also a requirement for #85, which uses the line time for the initial word.

[andy840119]

I've alwo never seen a mix of multiple line times and word times being used in the wild.

I know it's hard to have those case, but if it's still reasonable to keep both multiple line times and word times?

[Maxr1998]

I know it's hard to have those case, but if it's still reasonable to keep both multiple line times and word times?

I don't think it is, since I'd prefer to not parse them at all than to return wrong results. If the line isn't parsed, the user will notice their mistake and fix the LRC file.

[andy840119]

If that so, maybe tell the reason before if (startTimes.Length is 0 or > 1) why need to handle this condition.

e.g.

It's not possible to have multiple line times and word times.
If both case happened, for xxx reason, will drop the time-tag info.

[Maxr1998]

Tweaked the comment 👍

[andy840119]

I can give it a pass that drop the time-tag if has multiple time-tags.
But I'm not fully convinced, which future developers might have same issue.

Give adjust the comment a try ✊🏼

[andy840119]

Consider those edge not happened frequently, it's ok for me to handle with this way.