Last week I basically duplicated the serialization code to provide better debug output… today, I see this pass in my Mastodon feed. 😀 Well… what are the odds… most likely close to 100% according to how the universe seems to operate.

  • BB_C
    link
    fedilink
    arrow-up
    4
    ·
    3 months ago

    Last week I basically duplicated the serialization code to provide better debug output…

    Why was duplication needed, instead of implementing Debug using serialized output?

    And why would one want to rename stuff…etc in Debug anyway? It’s for debugging?

    This looks like purpose mismatch to me.

    • snaggenOP
      link
      fedilink
      arrow-up
      2
      ·
      3 months ago

      I wanted to use the debug fmt functions, to allow for pretty debug also.

      • BB_C
        link
        fedilink
        arrow-up
        3
        ·
        3 months ago

        I just looked at the code, and it’s actually a neat idea implementation wise, and I see now how it helped you.

        But I’m still of the opinion that its a purpose mismatch.