Last Comment Bug 676450 - Lots of mistakes in printf format in nsGtkIMModule.cpp
: Lots of mistakes in printf format in nsGtkIMModule.cpp
Product: Core
Classification: Components
Component: Widget: Gtk (show other bugs)
: unspecified
: All Linux
: -- normal (vote)
: mozilla8
Assigned To: Hiroyuki Ikezoe (:hiro)
Depends on:
  Show dependency treegraph
Reported: 2011-08-03 18:36 PDT by Hiroyuki Ikezoe (:hiro)
Modified: 2011-08-09 11:54 PDT (History)
3 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

Fix (4.82 KB, patch)
2011-08-03 18:36 PDT, Hiroyuki Ikezoe (:hiro)
masayuki: review+
Details | Diff | Splinter Review

Description User image Hiroyuki Ikezoe (:hiro) 2011-08-03 18:36:20 PDT
Created attachment 550577 [details] [diff] [review]

First, I found a mistake in the following:

336 void
337 nsGtkIMModule::OnFocusWindow(nsWindow* aWindow)
338 {
339     if (NS_UNLIKELY(IsDestroyed())) {
340         return;
341     }
343     PR_LOG(gGtkIMLog, PR_LOG_ALWAYS,
344         ("GtkIMModule(%p): OnFocusWindow, aWindow=%p, mLastFocusedWindow=%p",
345          this, aWindow));

So I've opened bug 676442, then found other mistakes in nsGtkIMModule.cpp.
Actually there are lots of mistakes in other files, I hope the others will be fixed after landing of the fix of bug 676442. This patch is just the beginning.

Note You need to log in before you can comment on or make changes to this bug.