>Bo Peng wrote:
>> - First, Entering "backgroundcolor=\color{red}" into the Listings
>> settings box will result in "backgroundcolor={\color{red}}" in the
>> output file. This makes the latex file uncompilable. To solve this I
>> just used the already defined, but unused, function "getParamValue"
in
>> InsetListingsParams.cpp to remove the {}'s.
>
>Maybe this is a listings version issue? I remember this worked before.
You're probably right. Today, at work (probably a newer version), it
seems to work quite well. In other words, it's time to figure this out
tonight, at home.
>Anyway, always removing {} is not a good idea because line 759 - 770
>serves some purpose and what you are doing simply undoes that. If you
>are sure {}should not be added in the first place, change 759-770.
>Otherwise, add cases in 759-770 when {} should not be added.
True, I was very reckless..
> Will color package be added only for global listings options?
Good point.
Things to fix:
- Listings version differences w.r.t. {}'s (as mentioned above).
- Sorting parameters fails (see also bugs 4884 and 5203):
Listings setting
backgroundcolor=\color{red},aboveskip=1em
is now converted into:
//obvious wrong (see also bug 4884)
aboveskip={1em}},
backgroundcolor={\color{red}
- Using \n in specifying unsorted parameters does not pass the
validation:
//not allowed according to validation
//leads to: Unknown listing parameter name: aboveskip=1em
backgroundcolor=\color{white}\n
aboveskip=1em
I read somewhere that Abdel might be willing / planning to change the UI
of this, but that it is not feasible for 1.6.0. However, I feel that the
things above should be resolved because a user will be disappointed if
he can't get it to work.
Thank you for your remarks.
Vincent
