Just some questions about minor improvements about markyp

[volfpeter]

Hi, thanks for the questions / observations.

1. The non-conditional space doesn't make the generated markup invalid. The recommended implementation also includes the extra space if the properties dictionary is empty.
2. The recommended implementation would raise a StopIteration exception if the items sequence is empty. That can of course be easily fixed and I also agree that this specific part could be better implemented. At the same time, the code works correctly.

And also, can i know why you create a copy of the separator object each time you yield it?

If separator is a callable, then it'll be called every time (as it should) and the return value will be added -- the callable itself can always return the same object, although in many cases you'll want different instances to add unique node keys for example, it depends on the use-case and the user. If separator is an ElementType instance, then sep will be a method that always returns that specific object (see line 41), there are no extra instantiations at all, since there's nothing to instantiate.

On the other hand, sep = lambda: separator() (line 30) could be simplified to sep = separator, the extra lambda is not necessary at all. This would be a tiny optimization though.

[volfpeter]

Closing for lack of activity.