RE: [patch] Shorten Windows path

2014-04-01 Thread Joey Ye
Ian, thanks for your comments. Please find answers and new version below: > -Original Message- > From: Ian Lance Taylor [mailto:i...@google.com] > Sent: 25 March 2014 21:09 > To: Joey Ye > Cc: gcc-patches > Subject: Re: [patch] Shorten Windows path > > On Tue,

RE: [patch] Shorten Windows path

2014-04-25 Thread Joey Ye
Ping > -Original Message- > From: Joey Ye [mailto:joey...@arm.com] > Sent: Tuesday, April 01, 2014 6:18 PM > To: 'Ian Lance Taylor' > Cc: gcc-patches > Subject: RE: [patch] Shorten Windows path > > Ian, thanks for your comments. Please

Re: [patch] Shorten Windows path

2014-04-25 Thread Ian Lance Taylor
Joey Ye [mailto:joey...@arm.com] >> Sent: Tuesday, April 01, 2014 6:18 PM >> To: 'Ian Lance Taylor' >> Cc: gcc-patches >> Subject: RE: [patch] Shorten Windows path >> >> Ian, thanks for your comments. Please find answers and new version below: >> >> &g

Re: [patch] Shorten Windows path

2014-05-08 Thread Jeff Law
On 04/25/14 04:16, Joey Ye wrote: Ping I've spoke with Kai a bit about this and he thinks it's appropriate and desirable to shorten paths on these kinds of filesystems. Ok for the trunk. Thanks and sorry for the wait, Jeff

Re: [patch] Shorten Windows path

2014-06-04 Thread Meador Inge
Hi, On 04/01/2014 05:17 AM, Joey Ye wrote: > diff --git a/libcpp/files.c b/libcpp/files.c > index 7e88778..ad68682 100644 > --- a/libcpp/files.c > +++ b/libcpp/files.c > @@ -387,8 +387,14 @@ find_file_in_dir (cpp_reader *pfile, _cpp_file *file, > bool *invalid_pch) >char *copy; >

RE: [patch] Shorten Windows path

2014-03-25 Thread Joey Ye
Ping > -Original Message- > From: Joey Ye [mailto:joey...@arm.com] > Sent: 19 February 2014 15:45 > To: gcc-patches@gcc.gnu.org; Ian Lance Taylor (i...@google.com) > Subject: [patch] Shorten Windows path > > Max length of path on Windows is 255, which is easy to exceed in a > complicated

Re: [patch] Shorten Windows path

2014-03-25 Thread Ian Lance Taylor
On Tue, Mar 25, 2014 at 1:58 AM, Joey Ye wrote: > Ping This code looks different on mainline. Writing "if ( do_canonical )" is not GCC style. This patch does not respect the configure option --disable-canonical-system-headers. Also I personally don't actually know what the consequences would b