Re: cleanup: include directives

2003-08-28 Thread Steven Edwards 
Your patch is broken. It didnt add the includes with Angle brackets.

--- Dimitrie O. Paun [EMAIL PROTECTED] wrote:
 This one has been in the queue long enough...
 
 ChangeLog
 Use angle brackets () rather than quotes () for the
 include directives in our .h files. This should avoid some
 potentially nasty surprises for Winelib apps.


__
Do you Yahoo!?
Yahoo! SiteBuilder - Free, easy-to-use web site design software
http://sitebuilder.yahoo.com

Re: cleanup: include directives

2003-08-28 Thread Jon Bright 
Steven Edwards wrote:

Your patch is broken. It didnt add the includes with Angle brackets.
It certainly looked to me like it did - are you sure your mailer didn't
interpret the mail as HTML, and the includes-in-angle-brackets as tags?
--
Jon Bright
Silicon Circus Ltd.
http://www.siliconcircus.com

Re: cleanup: include directives

2003-08-28 Thread Dimitrie O. Paun 
On Thu, 28 Aug 2003, Steven Edwards wrote:

 Your patch is broken. It didnt add the includes with Angle brackets.

No, it's not broken. I've 
  a) manually review the patch twice
  b) recompiled Wine
  c) as a quick check, it adds as many lines at it removes:

163 files changed, 542 insertions(+), 542 deletions(-)

-- 
Dimi.

Re: cleanup: include directives

2003-08-28 Thread Steven Edwards 
Jon Bright wrote:

It certainly looked to me like it did - are you sure your mailer didn't
interpret the mail as HTML, and the includes-in-angle-brackets as tags?
Your right it was my client. Time to upgrade.