[Libreoffice-commits] online.git: loolwsd/PROBLEMS
Tor Lillqvist
tml at collabora.com
Fri Apr 15 09:27:40 UTC 2016
loolwsd/PROBLEMS | 7 +++++++
1 file changed, 7 insertions(+)
New commits:
commit 6ca6bf0da68957b143d32c6d02a020830e7b5d76
Author: Tor Lillqvist <tml at collabora.com>
Date: Fri Apr 15 12:26:56 2016 +0300
Rant about over-use of variables where literals would work
diff --git a/loolwsd/PROBLEMS b/loolwsd/PROBLEMS
index 6f72e28..687fae9 100644
--- a/loolwsd/PROBLEMS
+++ b/loolwsd/PROBLEMS
@@ -23,4 +23,11 @@
to kill it with SIGKILL. Which of course leaves all the chroot jails
around.
+- There are lots of places where a std::string variable is defined,
+ initialised with a value, that is never changed. (In many cases it
+ is const, so could of course not be changed.) Then the variable is
+ used just once or twice, passed as a parameter to a function, or
+ used in a comparisin expression. This is fairly pointless and just
+ makes the code harder to read. Use string literals instead.
+
- ASCII art? Seriously?
More information about the Libreoffice-commits
mailing list