tag_class has to be a class.

[jjwatt]

Since we're targeting the clojure data format, why not support tag handlers through simple functions instead of requiring a TaggedElement class?

The nice thing is that if you only want to go in one direction, =edn_format.add_tag(myfunc, 'mytag')= already works.

It would be relatively simple to just take one or two functions instead of a class for handlers.

It would also be nice if the module functions like =edn_format.dumps= and =edn_format.loads= took optional functions as parameters for handlers.

[bfontaine]

Isn’t that already possible?

>>> from edn_format import add_tag, loads
>>> def my_verbose_tag(el):
...     print("Hey I got %s" % el)
...     return el
... 
>>> add_tag("verbose", my_verbose_tag)
>>> loads("#verbose 42")
Hey I got 42
42

[bfontaine]

It would also be nice if the module functions like =edn_format.dumps= and =edn_format.loads= took optional functions as parameters for handlers.

Do you mind making a pull-request?

I’m closing this since it doesn’t seem to be an issue. Please reopen if you think it is. Thanks!