Last Comment Bug 669299 - uneeded nsPrintfCString.h includes
: uneeded nsPrintfCString.h includes
Product: Core
Classification: Components
Component: Build Config (show other bugs)
: unspecified
: x86_64 Linux
-- normal (vote)
: mozilla8
Assigned To: arno renevier
: Gregory Szorc [:gps] (away until 2017-03-20)
Depends on:
  Show dependency treegraph
Reported: 2011-07-05 06:53 PDT by arno renevier
Modified: 2011-07-20 19:49 PDT (History)
1 user (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

patch v1 (16.27 KB, patch)
2011-07-05 06:54 PDT, arno renevier
no flags Details | Diff | Splinter Review
patch v1.1 (15.95 KB, patch)
2011-07-06 00:04 PDT, arno renevier
benjamin: review+
Details | Diff | Splinter Review

Description User image arno renevier 2011-07-05 06:53:31 PDT
nsPrintfCString.h defines only one class: nsPrintfCString. Therefore, when a file include nsPrintfCString.h but uses nsPrintfCString nowhere, it's unneeded.
There is currently the case in many files:
Comment 1 User image arno renevier 2011-07-05 06:54:18 PDT
Created attachment 543926 [details] [diff] [review]
patch v1
Comment 2 User image arno renevier 2011-07-06 00:04:52 PDT
Created attachment 544163 [details] [diff] [review]
patch v1.1

same patch with a commit message
Comment 3 User image :Ms2ger (⌚ UTC+1/+2) 2011-07-20 12:55:41 PDT
Btw, a commit message needs to include the bug number and the reviewer. Something like 

Bug 669299 - Remove unneeded nsPrintfCString.h includes; r=bsmedberg

for example.
Comment 4 User image Dão Gottwald [:dao] 2011-07-20 19:49:01 PDT

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