Closed Bug 673417 Opened 8 years ago Closed 8 years ago

imgLib cleanups

Categories

(Core :: ImageLib, defect)

x86
macOS
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla8

People

(Reporter: jrmuizel, Unassigned)

Details

Attachments

(2 files)

No description provided.
mLoadTime was added by stuart back in 2001. It hasn't as far as I can tell ever been used.
Attachment #547699 - Flags: review?(joe)
Attachment #547701 - Flags: review?(joe)
Comment on attachment 547701 [details] [diff] [review]
Kill more PRBools

>diff --git a/modules/libpr0n/src/RasterImage.cpp b/modules/libpr0n/src/RasterImage.cpp
>-PRBool
>+bool
> RasterImage::ShouldAnimate()
> {
>   return Image::ShouldAnimate() && mFrames.Length() >= 2 &&
>          !mAnimationFinished;
[...]
>diff --git a/modules/libpr0n/src/RasterImage.h b/modules/libpr0n/src/RasterImage.h
> protected:
>-  PRBool ShouldAnimate();
>+  bool ShouldAnimate();
> };

ShouldAnimate overrides an inherited method (from 'Image').  You'll need to also change the method in Image & VectorImage, or this will break method-overriding.
Comment on attachment 547699 [details] [diff] [review]
mLoadTime has never been used

Review of attachment 547699 [details] [diff] [review]:
-----------------------------------------------------------------

I only wish I could r- this
Attachment #547699 - Flags: review?(joe) → review+
Comment on attachment 547701 [details] [diff] [review]
Kill more PRBools

Review of attachment 547701 [details] [diff] [review]:
-----------------------------------------------------------------
Attachment #547701 - Flags: review?(joe) → review+
Comment on attachment 547701 [details] [diff] [review]
Kill more PRBools

Explicitly setting r-, to make sure Comment 3 gets addressed / noticed. :)
Attachment #547701 - Flags: review-
Target Milestone: --- → mozilla8
mwu has some mass boolification planned, so you probably don't need to do your own boolification patches.
(In reply to Daniel Holbert [:dholbert] from comment #6)
> Explicitly setting r-, to make sure Comment 3 gets addressed / noticed. :)

(/me notes that this was indeed addressed in cset d5db87177940.  Thanks! )
You need to log in before you can comment on or make changes to this bug.