Repository navigation
Quote removal should be configurable #29
Description
Activity
Actually, we perform quote removal only if all of the substitutions found in a word are solved. Do you think there is still need of an option to disable quote removal also when there are no subst?
I think they should be removed in all cases when possible also if not all of them has been replaced, because quotes only function is to work as delimiters and they have already done their purposse. On the other hand, a flag disabled by default to remove them is not bad, too.
quotes only function is to work as delimiters and they have already done their purpose
If the parser remove quotes even when it didn't perform substitution, the caller may not be able to perform them by itself, because you lose the information if the word or part of it are quoted (e.g. if it's quoted between single quotes, substitution should not occcur)
If expansion callback are not passed in options, quote removal should be avoid to allow a correct expansion on user side.