Last Comment Bug 794628 - Console is flooded and trashed with calFilterProperties debug messages
: Console is flooded and trashed with calFilterProperties debug messages
Status: RESOLVED FIXED
:
Product: Calendar
Classification: Client Software
Component: Internal Components (show other bugs)
: unspecified
: All All
: -- minor (vote)
: 2.0
Assigned To: Nobody; OK to take it and work on it
:
Mentors:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2012-09-26 14:25 PDT by Stefan Sitter
Modified: 2012-09-28 05:46 PDT (History)
1 user (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---


Attachments
reformat debug message to one line (20.96 KB, patch)
2012-09-26 14:25 PDT, Stefan Sitter
matthew.mecca: review+
Details | Diff | Review

Description Stefan Sitter 2012-09-26 14:25:25 PDT
Created attachment 665133 [details] [diff] [review]
reformat debug message to one line

I always have enabled calendar debug logging. For some time now the console is flooded and trashed with calFilterProperties debug messages like the following:

> [calFilterProperties] Applying filter:
>   start: 0
>   end: 3
>   status: 3
>   due: 15
>   category: null

The messages appear very often and take up a lot of space. This makes it difficult to see other and maybe more important messages.

Are the debug messages required at all? Or can they be removed? If we want to keep them the message should be at least reformatted to take up only 1 line instead of 6 lines.
Comment 1 Matthew Mecca [:mmecca] 2012-09-26 19:43:15 PDT
Comment on attachment 665133 [details] [diff] [review]
reformat debug message to one line

I haven't seen any issues reported with the new filtering code, so I think it's safe to remove these messages, r=mmecca to do that also.
Comment 2 Stefan Sitter 2012-09-28 05:46:19 PDT
Pushed to https://hg.mozilla.org/comm-central/rev/7b18f78f1ad9

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