199989 - Remove gNoisy code from libeditor.

Status: VERIFIED FIXED

(Core :: DOM: Editor, defect)

Description

[kinmoz]

Split off from bug 192569 where sfraser asked:

-  if (gNoisy) { printf("Editor::Do ----------\n"); }
+  if (gNoisy) { printf("Editor::DoTransaction ----------\n"); }

Can't we clean this up? This code may cause opt builds to link in the static
strings. Does anyone use it any more?

I never use this stuff, not sure if jfrancis does ... so perhaps it's time to
get rid of it.

Comment 1

[Bernd]

even if you dont remove it, you should probably #ifdef DEBUG it, this would fix
most of warren's build warnings
http://tinderbox.mozilla.org/SeaMonkey/warn1049133180.27326.html#warren

Comment 2

[timeless]

Attachment: round 1

i removed unused and buster gNoisys
i changed the others to NS_DEBUG/NS_DEBUG_EDITOR

Comment 3

[Kathleen :Brade]

Comment on attachment 122328 [details] [diff] [review]
round 1

r=brade
Changing sr to alecf since I know Kin is on vacation for 2 weeks.

Comment 4

[Alec Flett]

Comment on attachment 122328 [details] [diff] [review]
round 1

nice work! sr=alecf

Comment 5

[Joe Francis]

In answer to Kin's original question, no, I never used this stuff.  

Comment 6

[timeless]

.

Comment 7

[timeless]

checked in

Comment 8

[sairuh (rarely reading bugmail)]

rs vrfy.