[Pharo-dev] Writing jpeg files to disk

Usman Bhatti usman.bhatti at gmail.com
Tue Nov 5 05:54:19 EST 2013


Here is a slice the corrects the bug.
I cannot commit to pharo inbox.

Usman


On Tue, Nov 5, 2013 at 11:49 AM, Usman Bhatti <usman.bhatti at gmail.com>wrote:

> https://pharo.fogbugz.com/f/cases/12073/Writing-jpeg-files-to-disk
>
>
> On Tue, Nov 5, 2013 at 11:34 AM, Usman Bhatti <usman.bhatti at gmail.com>wrote:
>
>> Hello,
>>
>> Due to the changes in Pharo 3.0, I cannot write jpeg files on the disk.
>> This happens because when writing a jpeg file, we delete any precedent
>> version of the file to be exported. However, now DiskStore>>delete: throws
>> an exception when the file to be deleted is not present
>>
>> delete: path
>> | pathString encodedPathString |
>>  (self exists: path)
>>  ifFalse: [ ^ FileDoesNotExist signalWith: path ].
>>  ...
>>
>> Hence, we need to fix PluginBasedJPEGReadWriter>>putForm: quality:
>> progressiveJPEG: onFileNamed:  to not delete files not present.
>>
>> To reproduce the problem:
>> Display writeJPEGfileNamed: 'display.jpeg' progressive: true
>>
>> If confirmed, I'll open a bug report and propose a slice because
>> correction is quite simple ;)
>>
>> regards,
>> Usman
>>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.pharo.org/pipermail/pharo-dev_lists.pharo.org/attachments/20131105/3bfcc5ca/attachment-0002.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: Graphics-Files-usmanbhatti.48.mcz
Type: application/octet-stream
Size: 73743 bytes
Desc: not available
URL: <http://lists.pharo.org/pipermail/pharo-dev_lists.pharo.org/attachments/20131105/3bfcc5ca/attachment.mcz>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: SLICE-Issue-12073-Writing-jpeg-files-to-disk-usmanbhatti.1.mcz
Type: application/octet-stream
Size: 2609 bytes
Desc: not available
URL: <http://lists.pharo.org/pipermail/pharo-dev_lists.pharo.org/attachments/20131105/3bfcc5ca/attachment-0001.mcz>


More information about the Pharo-dev mailing list