Mailing-List: contact cygwin-apps-help AT cygwin DOT com; run by ezmlm Sender: cygwin-apps-owner AT cygwin DOT com List-Subscribe: List-Archive: List-Post: List-Help: , Mail-Followup-To: cygwin-apps AT cygwin DOT com Delivered-To: mailing list cygwin-apps AT cygwin DOT com Message-ID: <3CCAEEF5.9090406@ece.gatech.edu> Date: Sat, 27 Apr 2002 14:33:25 -0400 From: Charles Wilson User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:0.9.4) Gecko/20011019 Netscape6/6.2 X-Accept-Language: en-us MIME-Version: 1.0 To: Ralf Habacker CC: Kde-Cygwin , Cygwin-Apps , Binutils Subject: Re: cygwin ld import library issue fix (removing unused "_nm_" symbols) References: <000f01c1ee18$2c8739d0$d36707d5 AT BRAMSCHE> Content-Type: text/plain; charset=us-ascii; format=flowed Content-Transfer-Encoding: 7bit Ralf Habacker wrote: > > ... and accessing var0000 in client.cc > > Do you have really read the dll.h ? > > #define vars() \ > printf("v0000=%04d\n",var0000);\ > ^^^^^^^ > > the vars macro is called in client.cc !! sorry, I missed that var() was a macro. Thought it was a function. You're right; you've demonstrated that your patch doesn't break backwards compatibility. You've got my (tentative(*)) vote. --Chuck (*) tentative because I can't actually test it myself against HEAD, given the pre-existing problem with binutils HEAD on pe386.