* Peter Rosin wrote on Mon, Oct 04, 2010 at 10:02:15AM CEST:
> 
> I wrote a loop appending the first PATH component until the length
> exceeds the limit.  The longest possible PATH should be 499 characters
> this way, which seems OK to me, and it should have no "wild" components.

Cool.

> Den 2010-10-04 07:00 skrev Ralf Wildenhues:
> > * Peter Rosin wrote on Sun, Oct 03, 2010 at 08:28:47PM CEST:
> >>> This length doesn't yet trigger the compiler string literal length
> >>> limit; not sure whether it should?
> >>
> >> That's not reliable anyway as most compilers support so long strings
> >> that it's hard to tickle it.
> > 
> > FWIW, that is not necessary.  It would be sufficient if it were tickled
> > with the one compiler in question.
> 
> Since the compiler limit in this case is as large as 2048 (whoooa), which
> is about the same as you quoted for grep, I decided to not do that.

Good point.

The updated patch is fine.

Thanks,
Ralf

> Subject: [PATCH] cwrapper: split long lines when dumping the wrapper script.
> 
> * libltdl/config/ltmain.m4sh (func_emit_cwrapperexe_src): If
> the wrapper script contains long lines, split them for
> readability and to conform with C standards.
> * tests/cwrapper.at (cwrapper string length): New test, making
> sure we don't regress.

Reply via email to