386281 - "Type Manifest File:" spewed to console on launch.

Status: RESOLVED FIXED

(Core :: XPCOM, defect)

Description

[Justin Dolske [:Dolske]]

Comment 1

[Justin Dolske [:Dolske]]

Attachment: Remove printf

(doh)

Every time I launch Minefield, one of the first lines logged to console is:

Type Manifest File: /Users/dolske/[...profile path...]/xpti.dat

It's just one line, but lots of low-value just-one-lines are causing a lot of unneeded console spewage.

Comment 2

[Mark Banner (:standard8)]

Is this the very first line normally? I actually find it quite useful as it tells me which profile the app is starting up with (or even where it is if its a new profile) - which is very useful when you're trying to sort out profile migration and selection issues.

Anyway if you are going to get rid of it, you might as well just get rid of the whole ifdef.

Comment 3

[Justin Dolske [:Dolske]]

(In reply to comment #2)
> Is this the very first line normally? I actually find it quite useful as it
> tells me which profile the app is starting up with

I'm seeing it often with my builds, at least.

If we had common problems with things starting up with the wrong profile, then it would make sense to leave it in by default. But "this is sometimes useful" otherwise results in the mass console spewage we have today. :(

> Anyway if you are going to get rid of it, you might as well just get rid of the
> whole ifdef.

Oh, good point. Heh!

Comment 4

[Doug Turner (:dougt)]

Comment on attachment 270273 [details] [diff] [review]
Remove printf

it is a debug build, but i suppose that the location of the xpt file is a well know location.

agreed.

Comment 5

[Justin Dolske [:Dolske]]

Attachment: Remove the whole #ifdef

The previous r+ can probably carryover, but might as well be conservative. :)

Comment 6

[Doug Turner (:dougt)]

Comment on attachment 270381 [details] [diff] [review]
Remove the whole #ifdef

yeah, i guess I should have looked at where shown was being used...

Comment 7

[Justin Dolske [:Dolske]]

Checking in xpcom/reflect/xptinfo/src/xptiManifest.cpp;
/cvsroot/mozilla/xpcom/reflect/xptinfo/src/xptiManifest.cpp,v  <--  xptiManifest.cpp
new revision: 1.32; previous revision: 1.31
done