545780 - mimeiimg.h: update comment about supported types

Status: VERIFIED FIXED

(MailNews Core :: MIME, defect)

Description

[Serge Gautherie (:sgautherie)]

Bug 542770 comment 5:
{
 [e-mail]      2010-01-28 08:59:24 PST

/mailnews/mime/src/mimeiimg.h
Can't tell, sorry, but comment seems out of date (doesn't mention PNG etc.)
}

Probably remove 'image/xbm' and add newer ones.

Comment 1

[:aceman]

Currently there is:
"displayed inline (currently image/gif, image/jpeg, and image/xbm.)"

What exactly needs to be written there?

Comment 2

[Serge Gautherie (:sgautherie)]

(In reply to Serge Gautherie (:sgautherie) from comment #0)

> Probably remove 'image/xbm' and add newer ones.

Unless it would be acceptable to just remove the detailed list?

Comment 3

[:aceman]

I think that would be the most robust solution :)
I don't know if it is needed to have the formats listed in this place.

Comment 4

[:aceman]

Attachment: fix

Comment 5

[Andrew Sutherland [:asuth] (he/him)]

Comment on attachment 582066 [details] [diff] [review]
fix

Thanks for helping improve the comments!

Comment 6

[Mark Banner (:standard8)]

Checked in: http://hg.mozilla.org/comm-central/rev/7fa1fb316de0

Comment 7

[Serge Gautherie (:sgautherie)]

V.Fixed, per MXR.