How to start contributing code?
Sean Farley
sean.michael.farley at gmail.com
Fri Feb 6 23:08:41 UTC 2015
One of the reasons that you shouldn't drop the mailing list is so others
could try to help you when I get busy.
prafful minajigi writes:
> Hi...with respect to BUG 3718, I feel the line 363 in templater.py is
> partly responsible for the extra trailing spaces...
>
>
> 1. def join(context, mapping, args):
> 2. if not (1 <= len(args) <= 2):
> 3. # i18n: "join" is a keyword
> 4. raise error.ParseError(_("join expects one or two arguments"))
> 5. joinset = args[0][0](context, mapping, args[0][1])
> 6. if callable(joinset):
> 7. jf = joinset.joinfmt
> 8. joinset = [jf(x) for x in joinset()]
> 9. joiner = " "
> 10. if len(args) > 1:
> 11. joiner = stringify(args[1][0](context, mapping, args[1][1]))
>
> I feel this function is responsible for the Join operation. In the above
> code line 9 is causing the extra spaces to be generated ..
What? I'm certain that you have misread the bug report which is actually
suggesting using the 'join' template function to *add* a space between
iterated items.
More information about the Mercurial
mailing list