917269 - Remove kFontAppleMyungjo

Status: RESOLVED FIXED

(Core :: Graphics, defect)

Description

[(no longer active)]

Attachment: Patch (v1)

This variable is unused.

Comment 1

[Jonathan Kew [:jfkthame]]

Comment on attachment 805952 [details] [diff] [review]
Patch (v1)

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

Hmm. John, this comes from bug 705594. Did you intend to actually use this name?

Comment 2

[(no longer active)]

(In reply to Jonathan Kew (:jfkthame) from comment #1)
> Comment on attachment 805952 [details] [diff] [review]
> Patch (v1)
> 
> Review of attachment 805952 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> Hmm. John, this comes from bug 705594. Did you intend to actually use this
> name?

If yes, it would be nice to add it back in the patch which adds the usage?  This currently breaks builds with warnings as errors on newer clang versions.

Comment 3

[(no longer active)]

Note that this is now the only bug that breaks trunk builds with newer clang versions. :(

Comment 4

[John Daggett (:jtd)]

Comment on attachment 805952 [details] [diff] [review]
Patch (v1)

At this point, I can't say I remember... :P

Comment 5

[(no longer active)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/76b02f97bebc

Comment 6

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/76b02f97bebc