Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

JPEGReadWriter cannot read some files the PluginBasedJPEGReadWriter can #16534

Open
pavel-krivanek opened this issue Apr 27, 2024 · 0 comments · May be fixed by #16535
Open

JPEGReadWriter cannot read some files the PluginBasedJPEGReadWriter can #16534

pavel-krivanek opened this issue Apr 27, 2024 · 0 comments · May be fixed by #16535

Comments

@pavel-krivanek
Copy link
Collaborator

Bug description
Some files cannot be read using JPEGReadWriter while PluginBasedJPEGReadWriter opens them.

For example the following one:
devices

'devices.jpg' asFileReference binaryReadStreamDo: [ :stream |
(JPEGReadWriter on: stream) nextImage ].

'devices.jpg' asFileReference binaryReadStreamDo: [ :stream |
(PluginBasedJPEGReadWriter on: stream) nextImage ]

The first one fails with error Error: marker D0 cannot be handled.
Question is if it makes sense to keep them both in the image or rather rename PluginBasedJPEGReadWriter to JPEGReadWriter (which should be deleted or externalized) and deprecate the first name.

Version information:

  • Pharo Version: 11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant