diff options
author | Denys Vlasenko <vda.linux@googlemail.com> | 2009-08-03 17:57:59 +0200 |
---|---|---|
committer | Denys Vlasenko <vda.linux@googlemail.com> | 2009-08-03 17:57:59 +0200 |
commit | 1404fbfe4bf90d7f9dfe36deda484fe3e91b4472 (patch) | |
tree | a4ebd103d84e857fddae10030ff1746306f94e20 /src/Applet/Applet.cpp | |
parent | 3989e5b19a1472d270c7068b5c69f2eecf0ddeb4 (diff) | |
download | abrt-1404fbfe4bf90d7f9dfe36deda484fe3e91b4472.tar.gz abrt-1404fbfe4bf90d7f9dfe36deda484fe3e91b4472.tar.xz abrt-1404fbfe4bf90d7f9dfe36deda484fe3e91b4472.zip |
lib/Plugins/CCpp.cpp: move functions out of class where appropriate.
I was struggling to read the code where classes have member functions
with no apparent reasons to be such: they do not use any members
of the class.
Having them members of the class have only disadvantages:
they need to be declared in the class (thus you need to touch TWO files),
they cannot be shared among different classes,
they look confusing at callsites - the code falsely suggests
that function uses or alters object's state,
they also are a bit less efficient.
I was hesitating to do so before, but now I was told that
there is a plan to gradually move to C implementation...
then this change makes it a bit closer to that.
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
Diffstat (limited to 'src/Applet/Applet.cpp')
0 files changed, 0 insertions, 0 deletions