Max Schaefer 4825b6e5ad topic_list: Remove a spurious assignment and simplify a conditional. ...

Variable `show_topic` was assigned in both branches of the
conditional, but the assignment in the "then" branch was useless,
since the variable wasn't subsequently read. Hence the assignment can
be dropped, leaving the "then" branch empty. The "if" statement can
then be simplified by removing the "then" branch entirely and flipping
the condition. Since `show_topic` is now only used inside the "if", it
is slightly tidier (though semantically equivalent) to move its
declaration inside.

2017-08-22 23:49:47 -07:00

7.2 KiB

Raw Blame History

View Raw