diff options
author | Mathias Gumz <akira@fluxbox.org> | 2015-01-02 16:04:36 (GMT) |
---|---|---|
committer | Mathias Gumz <akira@fluxbox.org> | 2015-01-02 16:08:44 (GMT) |
commit | a3c0b049bf4fabee1b7b4ab3a5513dc11c5e9dd4 (patch) | |
tree | f42196489304515ce593161554904a455757e699 /data/Makemodule.am | |
parent | 0e8a27e931e66a4823b761983dca38a59fa6f37f (diff) | |
download | fluxbox-a3c0b049bf4fabee1b7b4ab3a5513dc11c5e9dd4.zip fluxbox-a3c0b049bf4fabee1b7b4ab3a5513dc11c5e9dd4.tar.bz2 |
Fix corruption of fbrun-history
This commit fixes issues #72 (brought up + different solution by Mattias
Guns; I received a similar patch by 'Nable 80' via ML and discussed the
issue in #fluxbox with 'Nable 80'), patch #73 (Mattias Guns) and finally
patch #162 (Ulrich Eckhardt; this commit is heavily based upon Ulrich's
work).
The original code was overly complex. It tried to avoid writing
bytes to the disk at the expense of comprehensibility and as a result it
was buggy. I looked at both patches from Mattias and 'Nable 80' which address
the bug with skipping entries in the history-file (my fault: incorrect use
of outfile.ignore(1, '\n')): They provided a proper fix for the problem
but I decided to use Ulrich's code since it improves the whole code by making
it a lot simpler.
So, kudos to all of you.
Diffstat (limited to 'data/Makemodule.am')
0 files changed, 0 insertions, 0 deletions