• Saganaki@lemmy.zip
    link
    fedilink
    arrow-up
    7
    arrow-down
    1
    ·
    1 day ago

    Occasionally I do that…but only because 500 of those lines are my comments explaining everything.

      • Saganaki@lemmy.zip
        link
        fedilink
        arrow-up
        3
        ·
        15 hours ago

        I’m aware. Not always feasible. For example, had to add a custom video capture solution that captures the last 30 seconds of a process for crash handling purposes.

        You most definitely need to do add that much comments explaining the mp4 box format along with the box “hierarchy” of what is being written. Add to that MFT (h264 encode) code…

        Basically, if anything, the comments are for me for when I look back at that code.

        • MonkderVierte@lemmy.zip
          link
          fedilink
          arrow-up
          1
          ·
          14 hours ago

          had to add a custom video capture solution that captures the last 30 seconds of a process for crash handling purposes.

          Commanded from above? This smells like a noob idea.

            • Jaycifer@piefed.social
              link
              fedilink
              English
              arrow-up
              1
              ·
              13 hours ago

              This is my attempt to translate: You had to add a feature like that ? Must have been an order from your boss. That seems like a feature someone rather inexperienced and unknowledgeable would request.

              • Saganaki@lemmy.zip
                link
                fedilink
                arrow-up
                4
                ·
                13 hours ago
                1. Legal has issues with using existing libraries (including MIT). Definitely idiotic, but I can’t control that.
                2. Idea was mine.
                3. Subprocess that captures parent process active video with a rolling buffer for crash handling purposes is absolutely necessary when trying to reproduce issues in development (Gamedev editor). If you think associating the last 15s prior to a crash with a mindump isn’t helpful for debugging, I don’t know what to tell you.