761019 - gfx/harfbuzz/src/hb-ot-shape-complex-indic.cc:620: error: comma at end of enumerator list

Status: RESOLVED FIXED

(Core :: Graphics: Text, defect)

Description

[Landry Breuil (:gaston)]

See summary, bug #758236 broke my openbsd/4.2.1 builds. See http://buildbot.rhaalovely.net/builders/comm-central-amd64/builds/383/steps/build/logs/stdio

Fixup in a few.

Comment 1

[Landry Breuil (:gaston)]

Attachment: Remove extra comma

Probably fixes the build for me.

Comment 2

[John Daggett (:jtd)]

Pushed to inbound
https://hg.mozilla.org/integration/mozilla-inbound/rev/699ded02ce5a

Comment 3

[Jonathan Kew [:jfkthame]]

Reported upstream.

Comment 4

[Landry Breuil (:gaston)]

http://hg.mozilla.org/mozilla-central/rev/1dc3d837c727

Comment 5

[(no longer active)]

Also https://hg.mozilla.org/mozilla-central/rev/699ded02ce5a (this seems like it was double landed on both inbound and central).

Comment 6

[Landry Breuil (:gaston)]

(In reply to Ehsan Akhgari [:ehsan] from comment #5)
> Also https://hg.mozilla.org/mozilla-central/rev/699ded02ce5a (this seems
> like it was double landed on both inbound and central).

Yes, sorry my fault, i screwed up here while doing my push for 761123 & 761136.

Comment 7

[(no longer active)]

(In reply to Landry Breuil (:gaston) from comment #6)
> (In reply to Ehsan Akhgari [:ehsan] from comment #5)
> > Also https://hg.mozilla.org/mozilla-central/rev/699ded02ce5a (this seems
> > like it was double landed on both inbound and central).
> 
> Yes, sorry my fault, i screwed up here while doing my push for 761123 &
> 761136.

No harm done!  I just mentioned it for future reference.  :-)