Trailing Carriage Returns about simple-yet-powerful-srt-subtitle-parser-cpp HOT 3 CLOSED

I'm thinking that this may be something that works slightly different on Linux and Windows?
I'm suggesting this because on Windows I never witnessed infile.eof() being true in SubRipParser::parse.

The bottom of this function now looks like this for me:

		else if (turn)
		{
			turn = 0;
			_subtitles.push_back(new SubtitleItem(subNo,start,end,completeLine));
			completeLine = timeLine = "";
		}

		if(infile.eof())    //insert last remaining subtitle
		{
			turn = 0;
			_subtitles.push_back(new SubtitleItem(subNo,start,end,completeLine));
		}
	}

	if (turn)
	{
		_subtitles.push_back(new SubtitleItem(subNo, start, end, completeLine));
	}

This works for me on Windows for the cases where I have any number of carriage returns after the last line of text in the SRT file, including 0.

But I don't have the possibility to test on Linux so I'm not proposing it as a fix, just a possible solution to consider if anybody else is witnessing the same symptom under Windows.

from simple-yet-powerful-srt-subtitle-parser-cpp.

Oh, this definitely sounds like a bug! Sorry for the super-late response.

Thanks for the report!

from simple-yet-powerful-srt-subtitle-parser-cpp.

Same bug : #7.

from simple-yet-powerful-srt-subtitle-parser-cpp.