> ; org-colview: Remove no-op fallback cond clause
> 
> * lisp/org-colview.el (org-columns--compute-spec): Remove the final
>   (t nil) clause; `cond' already returns nil when no clause matches.
> 
> Refactoring: Remove Dead Code.
> 
> No behavior change.

While I do not object this particular change, do note that explicit cond
clause is not always bad. Sometimes, it is used for clarity - if the nil
return value is intended as a part of function spec.

Of course, clarity is always subjective, but, sometimes, making the
intention clear is important, even if the code is redundant.

-- 
Ihor Radchenko // yantar92,
Org mode maintainer,
Learn more about Org mode at <https://orgmode.org/>.
Support Org development at <https://liberapay.com/org-mode>,
or support my work at <https://liberapay.com/yantar92>

Reply via email to