libchdr icon indicating copy to clipboard operation
libchdr copied to clipboard

Suggestion: Add ZStandard Codec Support

Open kcgen opened this issue 5 years ago • 8 comments

ZStandard Codec Support

  • Likely supported via LibZStandard
  • Pass-through of API settings for common ZStandard features such as: number of cores, block sizes, compression settings, and so on.
  • The API will allow emulator-specific and platform-specific settings to optimize the performance of of the library.

kcgen avatar Oct 23 '20 23:10 kcgen

This repo is a library for reading MAME's chd format. It doesn't have any influence over the chd format itself. For that, you'll need to open issues on https://github.com/mamedev/mame/.

Sanaki avatar Oct 24 '20 02:10 Sanaki

Thanks @Sanaki! (Closing)

kcgen avatar Oct 24 '20 03:10 kcgen

I'd imagine this applies to #34 and #36 as well. Feel free to correct me if I'm wrong though. EDIT: I read over them again, not actually sure it does.

Sanaki avatar Oct 24 '20 05:10 Sanaki

(Ref: https://github.com/mamedev/mame/issues/7386)

kcgen avatar Oct 24 '20 16:10 kcgen

Can this be revisited? MAME just added zstd support to chdman in this PR: https://github.com/mamedev/mame/pull/11827

InquisitiveCoder avatar Dec 11 '23 19:12 InquisitiveCoder

Owww that’s a HUGE win ! Yes definitely !

Le lun. 11 déc. 2023 à 20:41, InquisitiveCoder @.***> a écrit :

Can this be revisited? MAME just added zstd support to chdman in this PR: mamedev/mame#11827 https://github.com/mamedev/mame/pull/11827

— Reply to this email directly, view it on GitHub https://github.com/rtissera/libchdr/issues/35#issuecomment-1850765878, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAW4WBKIWJKXTPDG5ZR5MJ3YI5OYNAVCNFSM4S5ER6D2U5DIOJSWCZC7NNSXTN2JONZXKZKDN5WW2ZLOOQ5TCOBVGA3TMNJYG44A . You are receiving this because you are subscribed to this thread.Message ID: @.***>

rtissera avatar Dec 11 '23 19:12 rtissera

Should this issue be reopened until it's implemented?

malvarenga123 avatar Jan 09 '24 19:01 malvarenga123

Yes since MAME chaman now does it.

rtissera avatar Jan 09 '24 19:01 rtissera

I believe this was resolved in #106 and can be closed again.

Sanaki avatar Feb 02 '24 23:02 Sanaki

Yes, indeed, MAME team bit the bullet and implemented in chdman 0.262 It needs to be manually specified at compression time. Closing.

rtissera avatar Feb 03 '24 00:02 rtissera