Демка для IAIMPExtensionFileExpander

[DarkDrawKill]

Это сделает Expander через ExecuteInMainThread.

Видимо от ошибки декодера не избавиться

[Artem]

Видимо от ошибки декодера не избавиться

Почему?! В плейлист просто не будут попадать невалидные значения

[DarkDrawKill]

Почему?! В плейлист просто не будут попадать невалидные значения

Если вернуть ошибку, вылезает ошибка декодера. Либо вернуть S_OK и пустой лист.

[Artem]

Если вернуть ошибку, вылезает ошибка декодера. Либо вернуть S_OK и пустой лист.

Если вернуть ошибку - это будет значить, что Expander не может раскрыть этот файл. Т.е. движок плеера должен либо найти следующий Expander, либо добавить файл как есть).

[DarkDrawKill]

Если вернуть ошибку - это будет значить, что Expander не может раскрыть этот файл.

Значит я могу просто кинуть S_OK и пустой лист виртуальных файлов, но будет выводить ошибку через ExecuteInMainThread. Либо зарегистрировать декодер-пустышку, который будет просто генерировать ошибку.
Первый вариант выглядит элегантней, так как он не оставляет нераскрытый файл в плейлисте.

[Artem]

Значит я могу просто кинуть S_OK и пустой лист виртуальных файлов, но будет выводить ошибку через ExecuteInMainThread. Либо зарегистрировать декодер-пустышку, который будет просто генерировать ошибку.
Первый вариант выглядит элегантней, так как он не оставляет нераскрытый файл в плейлисте.

Я бы и ошибку не выводил. Пользователь может закинуть в плеер папку с кучей файлов и отойти от компа. Будет не очень хорошо, если процесс добавления файлов застопорится на показе окна с ошибкой. Как по мне, валидация должна происходить непосредственно при модификации файла - для этого большинство продвинутых редакторов текста имеют встроенные JSON-валидаторы.

[DarkDrawKill]

Пользователь может закинуть в плеер папку с кучей файлов и отойти от компа. Будет не очень хорошо, если процесс добавления файлов застопорится на показе окна с ошибкой.

А как по мне неудобно будет, ты узнаёшь через неделю две, что некоторые файлы не добавлены из-за ошибки. А ошибка может крыться не только в неправильном JSON (например в строке не правильно указан путь до трека). Подобное дебагать будет очень тяжело.

[Artem]

А как по мне неудобно будет, ты узнаёшь через неделю две, что некоторые файлы не добавлены из-за ошибки. А ошибка может крыться не только в неправильном JSON (например в строке не правильно указан путь до трека). Подобное дебагать будет очень тяжело.

Т.е. JSON-ы после написания никто не будет тестировать? Посмотрите как работает CUE. Плеер добавляет в плейлист ссылку на контейнер (CUE) + указывает номер трека в этом контейнере. Если путь к аудио файлу прописан неверно, то плеер выдаст ошибку "файл не найден" на момент обработки ошибки  IAIMPVirtualFile.CreateStream, если же сам CUE криво составлен - часть данных просто не будет загружена в плейлист, да, пользователя об этом никто не уведомит, но это на совести составителя CUE.

[DarkDrawKill]

Посмотрите как работает CUE.

В отличии от CUE, я подразумеваю, что JSON будет заполнять сам пользователь (поэтому я и выбрал его как довольно лёгкий формат). Среднестатистический пользователь, загрузивший свой JSON и и не получивший треки после, первым делом подумает, что проблема в плагине, а не в самом файле, потому что он привык, что программа оповещает об ошибке, если она случилась со стороны пользователя. И это только средний, не говоря об большинстве. Поэтому я думаю, что оповестить пользователя об ошибке будет правильней.

[Artem]

И это только средний, не говоря об большинстве

Среднестатистический пользователь знать не знает, что такое JSON и как правильно его заполнять. В лучше случае, за него это будет делать ИИ

[DarkDrawKill]

Среднестатистический пользователь знать не знает, что такое JSON и как правильно его заполнять.

Он хотя бы сможет легко в нём разобраться.

В лучше случае, за него это будет делать ИИ

Которые тоже заполняют иногда неправильно.

гзв Ладно сделаю по вашему, но ошибки отправлю в лог.
гзв2 Странный какой-то ServiceLog, у него отдельные методы для char* и wchar_t*, но в документации указана один метод с PChar

[Artem]

Которые тоже заполняют иногда неправильно.

Если это так принципиально, можно сделать отдельную кнопочку "для валидации" или вообще - свой редактор JSON-а.

гзв2 Странный какой-то ServiceLog, у него отдельные методы для char* и wchar_t*, но в документации указана один метод с PChar

Возможно, я не обновил документацию. Посмотрю на досуге.