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

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(+),

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.