459333 - Fix (leftover) regression from bug 199298

Status: RESOLVED FIXED

(MailNews Core :: Import, defect)

Description

[Serge Gautherie (:sgautherie)]

Bug 199298 comment 10
{
From  Serge Gautherie   2005-07-31 09:43:59 PDT

(In reply to comment #7)
> Created an attachment (id=164650) [edit] [details]
> proposed fix

|tPath| was removed, but the following line update was missed:
{{
617         MAPI_TRACE1( "~~ERROR~~ OpenStreamOnFile failed - temp path:
%s\r\n", tPath);
}}

Can you fix this too ?
}

Comment 1

[Serge Gautherie (:sgautherie)]

Attachment: (Av1) Make it compile with MAPI_DEBUG [Checkin: Comment 3]

[Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9.1b2pre) Gecko/20081009 SeaMonkey/2.0a2pre] (home, debug default) (W2Ksp4)

(compiled, but not run: should be trivial enough)

Comment 2

[David :Bienvenu]

Comment on attachment 342525 [details] [diff] [review]
(Av1) Make it compile with MAPI_DEBUG
[Checkin: Comment 3]

one nit - you don't need to cast string.get to a const char *; it already is one, isn't it?

Comment 3

[Serge Gautherie (:sgautherie)]

Comment on attachment 342525 [details] [diff] [review]
(Av1) Make it compile with MAPI_DEBUG
[Checkin: Comment 3]

http://hg.mozilla.org/comm-central/rev/c46cb5eed927
with comment 2 suggestion.

See
http://mxr.mozilla.org/mozilla-central/source/xpcom/glue/nsStringAPI.h?mark=786,847#750