Since the iteration over all the metaslots happens so many times, it turns out that the time in JsonObj.__bool__ is actually appreciable. A simple swap out with is None gives us a 137% perf boost for free :)
lots more perf to be had in this method in a fuller refactor but wanted to get this one in here since it's so uncontroversial, should be quick review
More free perf!
Since the iteration over all the metaslots happens so many times, it turns out that the time in
JsonObj.__bool__is actually appreciable. A simple swap out withis Nonegives us a 137% perf boost for free :)lots more perf to be had in this method in a fuller refactor but wanted to get this one in here since it's so uncontroversial, should be quick review
before (
__bool__in pink):after (
__bool__not present anymore):