diff options
author | Mathias Gumz <akira@fluxbox.org> | 2015-01-03 17:45:28 (GMT) |
---|---|---|
committer | Mathias Gumz <akira@fluxbox.org> | 2015-01-03 17:45:28 (GMT) |
commit | 65e06542491375953a481aee3f99ad412721b2d4 (patch) | |
tree | 7c151d92b8b1ca01270bf554b2031905405c50cc /src/ButtonTool.cc | |
parent | bd519dd95200aa432fb3b431a4244ffb25c5de93 (diff) | |
download | fluxbox-65e06542491375953a481aee3f99ad412721b2d4.zip fluxbox-65e06542491375953a481aee3f99ad412721b2d4.tar.bz2 |
Fix typo / editor bug or something like that.
While running code analysis tools this issue popped up. As it is written it is
clearly a NOP, but the reason for why it is written that way remains unknown.
I suspect that it was a textfragment introduced by some repeat-function of the
used editor (think '.' in VIM). The code was introduced by commit a932a7a801
and looked like this in the original form:
if(!XQueryTree(FbTk::App::instance()->display(), window().window(),
&root_return, &parent_return,
&children_return, &nchildren_return))
parent_return=parent_return;//return;
Look at the outcommented return statement. I decided to return from that
function instead of the NOP.
Diffstat (limited to 'src/ButtonTool.cc')
0 files changed, 0 insertions, 0 deletions