Ben Kurtovic
Ben Kurtovic
The bug makes sense based on how `Template.add` works. At the moment, it only tries to match whitespace before and after the parameter name and value based on the most...
I partially fixed it—the new param does get placed on the correct line—but it's not perfect. I would like to revisit this algorithm in the future at some point. (Implementation...
The inconsistency is fixed in the above commit, but you've pointed out another thing that needs a more involved fix, which I'll have to work on later.
Actually, @jfolz, I think you might be right. I recall that the overhead of linked lists (both in terms of memory usage and their conflicts with CPU caching, etc) often...
You're right, that's missing functionality. Will add it for 0.5.
I don't think the additional complexity of a `strip_whitespace` flag or whatever is worth it... this would just make things more confusing. In general, you should be fine using `.strip()`....
TBH, I don't see the advantage of one order over the other. Can you give an example?
The idea of parsing modes is something I am considering for v0.5/1.0. It seems like the best solution to this issue and related ones like #100.
Leaving this open due to the surprising (and somewhat upsetting) external link difference, but the template/heading conflict should be resolved.