XYsubfilter uses the default (Arial) style instead of the file

[MatiasMovie]

XYsubfilter uses the default (Arial) style instead of the file
Tested with MPC-HC and MPC-BE, subs work fine in MPV player (libass)
I think it's problem with "space" on the end in style name

///EDIT///
I changed the video to a random light sampel and added problematic subtitles (Now there is only 27.9 MB)
https://gofile.io/d/RqiFs6

[pinterf]

1.2G? Don't you have only the extracted sub file?
Or you can test with the current build I've just published (mainly because I made changed to the Avisynth filter), if the problem still occurs.

[realfinder]

here the sub file
[shisha] Ishuzoku Reviewers - 01 [BD 1080p x265 FLAC] [954416D9]_track3_pol.zip

[MatiasMovie]

Unfortunately, the error occurs only if the subtitles are in the MKV file, external works well
I am using version 3.2.0.805

v3.2.0.804 (20210306) still have problem

[MatiasMovie]

XySubFilter and external subtitles:

XySubFilter and mkv:

mpv and mkv:

[pinterf]

Thanks for the report, fixed in f71eb96
Can you build it yourself or I should prepare a test build for you?

[realfinder]

I did test the new sample in mpc with it's internal sub render (I have no xy-VSFilter nor VSFilter in that system) and it's same problem, maybe it's mpc bug after all

[MatiasMovie]

Thanks for the report, fixed in f71eb96
Can you build it yourself or I should prepare a test build for you?

I do not know how to build it, but I can wait for a new version.

[pinterf]

Check this one
https://drive.google.com/uc?export=download&id=1yDCrMbaULP3h2yrdNZE3OCoH7g__wP4z

[MatiasMovie]

Now the subtitles do not display
Tested in MPC-HC and MPC-BE

[pinterf]

Sorry, then this one should work, I hope.
https://drive.google.com/uc?export=download&id=1ZvYtb3l7UeJ0Xz3BgTj9X2x6q_W_O5eR

[MatiasMovie]

now works good with internal subs, the information is probably displays a bad version number

[MatiasMovie]

I found a problem, now external subtitles have a problem with this style in MPC-HC/BE

FPS_test_1080p23.976_L4.1.zip

[pinterf]

Great, thanks for checking that

[astiob]

Please revert f71eb96. No other ASS renderer does this, and it actually breaks this file as you can see in the latest comment by OP.

But also, please report and fix bugs upstream in https://github.com/Cyberbeing/xy-VSFilter. We have too many forks as it is.

The problem is in CSubtitleInputPin:

xy-VSFilter/src/subtitles/SubtitleInputPin.cpp

Lines 260 to 265 in 13346cb

	Explode(str, sl, ',', fields);
	if(sl.GetCount() == fields)
	{
	stse.readorder = wcstol(sl.RemoveHead(), NULL, 10);
	stse.layer = wcstol(sl.RemoveHead(), NULL, 10);
	stse.style = sl.RemoveHead();

https://github.com/Cyberbeing/xy-VSFilter/blob/fc01a8da5ea6af9091aaab839bc62dc94a90094e/src/subtitles/SubtitleInputPin.cpp#L260-L265

which uses Explode, which trims every field:

xy-VSFilter/src/dsutil/text.h

Lines 35 to 38 in 13346cb

	sl.AddTail(str.Mid(i).Trim());
	break;
	} else {
	sl.AddTail(str.Mid(i, j - i).Trim());

For what it’s worth, this also affects MPC-HC’s internal subtitle renderer.

[pinterf]

Thanks astiob, then it wasn't the way to go.
As for putting all my changes upstream: if a change is worth to be put there they're gonna cherry pick as it happened in the past. Once I'd do a PR all my changes would appear there in a long row. Unless I change my habit of working into a single branch and make a different branch for each feature/fix. But this project was/is so dead that I really didn't think they would spend any time on with my modifications. Or it breaks the project rules so much that they get incompatible with the original Cyberbeing branch. (E.g. if I make it to require c++17 which is a highly logical thing in 2021)

[pinterf]

Next iteration, please.
https://drive.google.com/uc?export=download&id=14RG8OL-YTybEj4BlOPKCaOCEE2Bwkt0b

[MatiasMovie]

Now works well