-
Notifications
You must be signed in to change notification settings - Fork 3
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
Simplify FLIF pass 1 #77
Conversation
- Keep as much of the FLIF stuff inside `flif->list` as possible - Needs cleanup of a bunch of old comments - Aborting decoding only necessary for progressive decoding
base.rkt
Outdated
@@ -83,9 +83,9 @@ | |||
; number of times to loop a FLIF (0 = forever) | |||
(define image-num-loops 0) | |||
(define animation-thread (make-parameter #f)) | |||
(define decoder-thread (make-parameter #f)) | |||
;(define decoder-thread (make-parameter #f)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If these actually aren't used anywhere (and I see they're not) we prolly wanna delete the commented-out lines (unless you're planning to revive them very soon).
base.rkt
Outdated
(define bitmap (make-object bitmap% width height #f #t)) | ||
(send bitmap set-argb-pixels 0 0 width height pixels) | ||
bitmap)) | ||
;(define width (flif-image-get-width image)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, kill these lines. Could get confusing when casually perusing the code with these values "defined" in two places right by each other.
want-animation? passed as an optional keyword Get rid of bizarre parameterize in thumbnail generation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, this looks pretty good. There's some stuff that feels a little awkward to me, but I think that's just me coming from other programming languages.
flif->list
as possible