Hi Jan,
First - great to have you working on this =)
On 17/03/17 10:33, Jan Iversen wrote:
3) the file Kit/Kit.cpp contains a long cascade of if else statements,
because commands are communicated as strings.
is this on purpose ? why not have an Kit_enum.hpp that is converted
to Kit_enum.js and then have the commands be numbers.
It is one possible optimization; but of course - it is quite nice to
have general code where we can add commands without touching all the
intermediate message multiplexers =)
The advantage
would be we can use simple switch statements which are a lot faster (and
looking at the performance during the esc-meeting, we could do with
better performance).
I -think- performance issues are related to multiplexing, message
prioritizing, managing buffering, and a host of other exciting issues;
(until recently perhaps lock contention too) - at least CPU time doesn't
dominate the profiles I've seen either for the Kit prisoner or WSD.
So in general, while I love optimization, we should profile heavily
before optimizing I think - and follow the profiles =) Of course with a
different form-factor and set of profiling tools we'll perhaps get some
new use-cases to optimize for your use-case; but even so - I'd like to
see a profile first personally.
Thanks !
Michael.
--
michael.meeks@collabora.com <><, Pseudo Engineer, itinerant idiot
Context
Privacy Policy |
Impressum (Legal Info) |
Copyright information: Unless otherwise specified, all text and images
on this website are licensed under the
Creative Commons Attribution-Share Alike 3.0 License.
This does not include the source code of LibreOffice, which is
licensed under the Mozilla Public License (
MPLv2).
"LibreOffice" and "The Document Foundation" are
registered trademarks of their corresponding registered owners or are
in actual use as trademarks in one or more countries. Their respective
logos and icons are also subject to international copyright laws. Use
thereof is explained in our
trademark policy.