277121 - Remove printf on non DEBUG builds

Status: RESOLVED FIXED

(Core :: Printing: Output, defect)

Description

[Pete Collins (MDG)]

The lxr url posted above points to a stray printf which shouldn't be displayed
in non debug builds.

Patch forthcoming ...

Comment 1

[Pete Collins (MDG)]

Attachment: gfx/src/ps/nsAFMObject.cpp

Comment 2

[Jan Varga [:janv]]

Comment on attachment 170351 [details] [diff] [review]
gfx/src/ps/nsAFMObject.cpp

r=varga, it should pretty safe to remove this printf in non-debug builds :)

Comment 3

[Pete Collins (MDG)]

Thanks Jan, any buddies who can super review this?

Comment 4

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 170351 [details] [diff] [review]
gfx/src/ps/nsAFMObject.cpp

sr=bzbarsky

Comment 5

[Pete Collins (MDG)]

As soon as Myk updates my new id_dsa.pub, I'll check this in.

Anyone else please feel free to check it in ...

Comment 6

[Pete Collins (MDG)]

Checked in ...

Thanks all

Comment 7

[Christian :Biesinger (don't email me, ping me on IRC)]

the tree is frozen currently, btw.

Comment 8

[Kenneth Herron]

This is fixed, isn't it?

Comment 9

[Pete Collins (MDG)]

Yup

Comment 10

[Pete Collins (MDG)]

This patch needs to land on the branch, how do I do that?