> -----Original Message----- > From: William A. Rowe, Jr. [mailto:[EMAIL PROTECTED] > Sent: Monday, October 29, 2007 11:30 PM > To: stdcxx-dev@incubator.apache.org > Subject: Re: svn commit: r588290 - > /incubator/stdcxx/branches/4.2.x/etc/config/windows/projectdef.js > > > [EMAIL PROTECTED] wrote: > >> Author: faridz > >> Date: Thu Oct 25 10:40:21 2007 > >> New Revision: 588290 > >> > >> URL: http://svn.apache.org/viewvc?rev=588290&view=rev > >> Log: > >> 2007-10-25 Farid Zaripov <[EMAIL PROTECTED]> > >> > >> * projectdef.js (projectCreateVCProject): Generate .pdb file > >> in $(OutDir) instead of $(IntDir). Use common .pdb file > >> instead of two (compiler's and linker's). > > Are you sure you didn't mean to continue inverting these, > therefore storing the compiler's pdb results in the IntDir? > > Yes, the resulting link-output pdb must reside in OutDir as > you have patched, but combining them may lead to some > unstable results and also bloated .pdb's without benefits to > the debugger (?).
I don't completely understand which one unstable results could be? Before the patch: 1. in static builds the source pdb was generated in IntDir with the name vc71.pdb for MSVC 7.1 (or vc80.pdb for MSVC 8.0); the binary pdb doesn't exist since the static library is produced by lib.exe, but not by link.exe. 2. in dynamic builds the source pdb was generated in IntDir with the name vc71.pdb for MSVC 7.1 (or vc80.pdb for MSVC 8.0); the binary pdb was generated in OutDir with the name libstdxx.pdb. After the patch: 1. in static builds the source pdb is generating in OutDir with the name libstdxx.pdb; the binary pdb doesn't exist since the static library is produced by lib.exe, but not by link.exe. 2. in dynamic builds the source pdb and the binary pdb both generating in OutDir using the single file with the name libstdxx.pdb. I suppose you mean that the information from the source pdb in the dynamic builds will be unuseful for the library users because they are use the binary .dll file, but not the .obj files. And adding this information to the binary pdb just increase the pdb file without any benefits. Right? So you propose to revert the patch only for dynamic builds? Farid.
