Re: cygstart patch

2005-03-08 Thread Michael Schaap
On 8-Mar-2005 6:21, Charles Wilson wrote: Michael Schaap wrote: This time with patch. :-[ Applied. Thanks, Chuck! Michael -- Unsubscribe info: http://cygwin.com/ml/#unsubscribe-simple Problem reports: http://cygwin.com/problems.html Documentation: http://cygwin.com/docs.html

Re: cygstart patch

2005-03-07 Thread Michael Schaap
Hi Anthony, Chuck, all, On 4-Mar-2005 4:21, Charles Wilson wrote: Anthony -- I've applied this patch. Thanks! Sorry for jumping in so late... While I agree that the revised patch looks good and does the job, I'm not too happy with the constant reallocation that's going on. (I know, there won't

Re: cygstart patch

2005-03-07 Thread Michael Schaap
This time with patch. :-[ Michael On 8-Mar-2005 2:13, Michael Schaap wrote: Hi Anthony, Chuck, all, On 4-Mar-2005 4:21, Charles Wilson wrote: Anthony -- I've applied this patch. Thanks! Sorry for jumping in so late... While I agree that the revised patch looks good and does the job, I'm not

Re: cygstart patch

2005-03-07 Thread Charles Wilson
Michael Schaap wrote: This time with patch. :-[ Applied. -- Chuck -- Unsubscribe info: http://cygwin.com/ml/#unsubscribe-simple Problem reports: http://cygwin.com/problems.html Documentation: http://cygwin.com/docs.html FAQ: http://cygwin.com/faq/

Re: cygstart patch

2005-03-04 Thread Charles Wilson
Alec Klauk wrote: I have unsubscribed 10 times to this mailing. How do you do it? First, you DON'T pick some random schmoe on the list and ask him. Second, you read and follow the instructions... http://cygwin.com/lists.html#subscribe-unsubscribe http://sourceware.org/lists.html#faqs (especially

RE: cygstart patch

2005-03-03 Thread Derosa, Anthony CIV NAVAIR 2035, 2, 205/214
To: cygwin@cygwin.com Subject: Re: cygstart patch Derosa, Anthony CIV NAVAIR 2035, 2, 205/214 wrote: I found a small bug and added a feature to the cygstart utility, which is part of the cygutils package. The feature that I added removes the limit on the length of the command line

RE: cygstart patch

2005-03-03 Thread Dave Korn
Original Message From: Charles Wilson Sent: 03 March 2005 06:59 Derosa, Anthony CIV NAVAIR 2035, 2, 205/214 wrote: I found a small bug and added a feature to the cygstart utility, which is part of the cygutils package. The feature that I added removes the limit on the length of the

RE: cygstart patch

2005-03-03 Thread Derosa, Anthony CIV NAVAIR 2035, 2, 205/214
: cygstart patch Original Message From: Charles Wilson Sent: 03 March 2005 06:59 Derosa, Anthony CIV NAVAIR 2035, 2, 205/214 wrote: I found a small bug and added a feature to the cygstart utility, which is part of the cygutils package. The feature that I added removes

RE: cygstart patch

2005-03-03 Thread Igor Pechtchanski
On Thu, 3 Mar 2005, Derosa, Anthony CIV NAVAIR 2035, 2, 205/214 wrote: What do you think about Chuck's concerns regarding strcat() vs. strncat()? I'm no expert either, but usually the purpose of strncat() is to avoid a buffer overrun when copying an arbitrary-sized string into a

RE: cygstart patch

2005-03-03 Thread Derosa, Anthony CIV NAVAIR 2035, 2, 205/214
, March 03, 2005 9:34 To: Derosa, Anthony CIV NAVAIR 2035, 2, 205/214 Cc: cygwin@cygwin.com Subject: RE: cygstart patch On Thu, 3 Mar 2005, Derosa, Anthony CIV NAVAIR 2035, 2, 205/214 wrote: What do you think about Chuck's concerns regarding strcat() vs. strncat()? I'm no expert

RE: cygstart patch

2005-03-03 Thread Dave Korn
Original Message From: Derosa, Anthony CIV NAVAIR 2035, 2, 205/214 Sent: 03 March 2005 14:22 Dave, you're right! I was forgetting the NUL in realloc. I'm surprised that the original fix has been working for me. What do you think about Chuck's concerns regarding strcat() vs.

RE: cygstart patch

2005-03-03 Thread Derosa, Anthony CIV NAVAIR 2035, 2, 205/214
But there is an important _process_ problem. It is the number one sin of CVS to checkin anything other than ***exactly*** the code that you compiled and tested, so you aren't excused from having to re-compile and re-test the code with the +1 = +2 modification applied to it, and at that

Re: cygstart patch

2005-03-03 Thread Charles Wilson
Anthony -- I've applied this patch. Thanks! -- Chuck Derosa, Anthony CIV NAVAIR 2035, 2, 205/214 wrote: But there is an important _process_ problem. It is the number one sin of CVS to checkin anything other than ***exactly*** the code that you compiled and tested, so you aren't excused from

cygstart patch

2005-03-02 Thread Derosa, Anthony CIV NAVAIR 2035, 2, 205/214
I found a small bug and added a feature to the cygstart utility, which is part of the cygutils package. The feature that I added removes the limit on the length of the command line arguments passed to the target application, which was previously limited to MAX_PATH. The bug I fixed was in

Re: cygstart patch

2005-03-02 Thread Christopher Faylor
On Wed, Mar 02, 2005 at 10:47:40AM -0500, Derosa, Anthony CIV NAVAIR 2035, 2, 205/214 wrote: I found a small bug and added a feature to the cygstart utility, which is part of the cygutils package. The feature that I added removes the limit on the length of the command line arguments passed to

cygstart patch

2005-03-02 Thread Derosa, Anthony CIV NAVAIR 2035, 2, 205/214
I found a small bug and added a feature to the cygstart utility, which is part of the cygutils package. The feature that I added removes the limit on the length of the command line arguments passed to the target application, which was previously limited to MAX_PATH. The bug I fixed was in

Re: cygstart patch

2005-03-02 Thread Charles Wilson
Derosa, Anthony CIV NAVAIR 2035, 2, 205/214 wrote: I found a small bug and added a feature to the cygstart utility, which is part of the cygutils package. The feature that I added removes the limit on the length of the command line arguments passed to the target application, which was previously