Karthik Nayak <karthik.188@xxxxxxxxx> writes:
> The 'ifexists' atom allows us to print a required format if the
> preceeding atom has a value. If the preceeding atom has no value then
> the format given is not printed. e.g. to print "[<refname>]" we can
> now use the format "%(ifexists:[%s])%(refname)".
A handful of "huh?" on the design.
- The atom says "if *exists*" and explanation says "has a value".
How are they related? Does an atom whose value is an empty
string has a value? Or is "ifexists" meant to be used only to
ignore meaningless atom, e.g. %(*objectname) applied to a ref that
refers to an object that is not an annotated tag?
- That %s looks ugly. Are there cases where a user may want to say
%(ifexists:[%i]) or something other than 's' after that per-cent?
. Is it allowed to have more than one %s there?
. Is it allowed to have no %s there?
- The syntax makes the reader wonder if [] is part of the
construct, or just an example of any arbitrary string, i.e. is
"%(ifexists:the %s can be part of arbitrary string)" valid?
- If an arbitrary string is allowed, is there any quoting mechanism
to allow ")" to be part of that arbitrary string?
- What, if anything, is allowed to come between %(ifexists...) and
the next atom like %(refname)? For example, are these valid
constructs?
. %(ifexists...)%(padright:20)%(refname)
. %(ifexists...) %(refname) [%(subject)]
- This syntax does not seem to allow switching on an attribute to
show or not to show another, e.g. "if %(*objectname) makes sense,
then show '%(padright:20)%(refname:short) %(*subject)' for it".
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html