RFR v5 - 8027634: Support @argfiles for java command-line tool
mandy.chung at oracle.com
Mon Aug 17 21:04:21 UTC 2015
On 08/17/2015 08:00 AM, Henry Jen wrote:
> On Aug 17, 2015, at 7:10 AM, Henry Jen <henry.jen at oracle.com> wrote:
>> I think I may be talking at a different thing, guess you meant the make JLI_List JLI_PreprocessArg(const char*) to void JLI_PreprocessArg(JLI_List, const char*) and pass in a list to hold all the arguments.
>> That is reasonable. Considered that but didn’t do it eventually because not much benefits and hide the “expansion” fact. Since now you mentioned this, it probably worth to do it that way.
> Now I remember another reason I didn’t do it, because of the wildcard processing in cmdtoargs.c. I don’t want to convert every platform to use StdArg or maintain different version of PreprocesArgs.
My suggestion is to keep JLI_PreprocessArg(const char* arg) is to return
a non-null JLI_List. If the given arg is not @argfile, you return a
static preallocated JLI_List with one single element and element =
The change in main.c and cmdtoargs.c add the arguments in either a
JLI_List or StdArg. For the non- at argfile case, you need to do
JLI_StringDup explicitly. For @argfile case, JLI_StringDup is done
before it returns JLI_List. I also suggest to add
JLI_FreeList(JLI_List*, boolean shallow_free) that should special case
the static preallocated JLI_List to clear the count and element and not
to free that list.
I think that will simplify the code.
More information about the core-libs-dev