pixelscript , 7 months ago

It’s a simple function definition that’s equivalent to:

<span style="color:#323232;">function confirm(value)
</span><span style="color:#323232;">{
</span><span style="color:#323232;">    if (value == true)
</span><span style="color:#323232;">    {
</span><span style="color:#323232;">        return true;
</span><span style="color:#323232;">    }
</span><span style="color:#323232;">    else
</span><span style="color:#323232;">    {
</span><span style="color:#323232;">        return false;
</span><span style="color:#323232;">    }
</span><span style="color:#323232;">}
</span>

Not the most original punchline; I’m sure you’ve seen it before. We were just baffled to actually see it in the wild.

Judging from the way this function was used, there no evidence to suggest it ever contained extra logic that was refactored out over time. I’m wholly convinced someone wrote this as-is and thought it was okay. I also knew that there’s no way this was extracted for DRY purposes, as it was only called in one place, and the rest of the codebase was extremely allergic to DRY.

It was also formatted like complete garbage. Indentation level was not consistent line by line. And, presumably due to some carelessness handling line endings, the entire code file developed double-spacing. Somehow it was checked into version control in that state.

All these little nits, from the code’s utter uselessness to its appalling formatting, compelled us to preserve it. It was like the entire rest of the shitty codebase in microcosm.