0007223: [RFE] GIFReadWriter and Animated Gifs can be done better.

Viewing Issue Simple Details [ Jump to Notes ]

[ View Advanced ] [ Issue History ] [ Print ]

Category

Severity

Reproducibility

Date Submitted

Last Update

0007223

[Squeak] Graphics

minor

N/A

11-14-08 03:39

Reporter

wiz

View Status

public

Assigned To

andreas

Priority

normal

Resolution

open

Status

assigned

Product Version

Summary

0007223: [RFE] GIFReadWriter and Animated Gifs can be done better.

Description

GIFReadWriter had a serious bug affecting the reading of animated gifs.

That has been diagnosed and patched in 0007210.

In the course of looking into that problem, I learned enough about gifs to wish for a more elegant reader.

Additional Information

The gif spec just about describes how to structure a program.
Also the current reader/writer has a high code smell due to too many ivars and methods with high temp var counts as well.

I am sure some of the justification for that was the need to coax speed out of the reader.

I think with a second pass over the code some simple resturcturing could make it more robust and compartmentalized.

I didn't want to burden the previous report with that task.

So I have started this one to contain the more extensive work.

Attached Files

Relationships

parent of	0007210	assigned	andreas	[Bug] Gif anim reader has problem reading certain animes.
child of	0007209	assigned	andreas	A Mother for animation bugs
Not all the children of this issue are yet resolved or closed.

There are no notes attached to this issue.

Issue History
Date Modified	Username	Field	Change
11-14-08 03:39	wiz	New Issue
11-14-08 03:39	wiz	Status	new => assigned
11-14-08 03:39	wiz	Assigned To	=> andreas
11-14-08 03:39	wiz	Relationship added	parent of 0007210
11-16-08 19:52	wiz	Relationship added	child of 0007209

Mantis 1.0.8[^]

40 total queries executed.
32 unique queries executed.