In addition to what I previously said at #19 about having this particular functionality in a separate serializer class... what's the deal with the argument here? An indentation string?!? Why? Why not have it be an integer representing number of an indentation string (specified with an additional option; space by default)?