BString::Split doesn't work as expected

[Janus]

Based on this issue ​https://github.com/HaikuArchives/Tipster/issues/1 I have investigated the problem... I think there is an error in the split function

499 bool
500 BString::Split(const char* separator, bool noEmptyStrings,
501	BStringList& _list) const
502{
503	int32 separatorLength = strlen(separator);
504	int32 length = Length();
505	if (separatorLength == 0 || length == 0 || separatorLength > length) {
506		if (length == 0 && noEmptyStrings)
507			return true;
508		return _list.Add(*this);
509	}
510
511	int32 index = 0;
512	for (;;) {
513		int32 endIndex = index < length ? FindFirst(separator, index) : length;
514		if (endIndex < 0)
515			endIndex = length;
516
517		if (endIndex > index || !noEmptyStrings) {
518			BString toAppend(String() + index, endIndex - index);
519			if (toAppend.Length() != endIndex - index
520				|| !_list.Add(toAppend)) {
521				return false;
522			}
523		}
524
525		if (endIndex == length)
526			break;
527
528		index = endIndex + 1;
529	}
530
531	return true;
532}

Line 528 should be

index = endIndex + separatorLength; 

The code works only if the separator is a single char...

I cannot compile Haiku someone can verify thanks.

[korli]

Fixed in ​https://git.haiku-os.org/haiku/commit/?id=a25f7264b6a3e66687df4cd9850dd7d3bb1243a7

Thanks!

[pulkomandy]

An unit test would be welcome, too :)

[korli]

Replying to PulkoMandy:

An unit test would be welcome, too :)

See ​https://git.haiku-os.org/haiku/diff/src/tests/kits/support/bstring/StringSplitTest.cpp?id=a25f7264b6a3e66687df4cd9850dd7d3bb1243a7