surensp...@gmail.com wrote: > I shall touch base with you on the IRC and will implement all the > tweaks as you suggest. If there are any best practices wiki link, > please guide me to that :) > > > There are one or two other things that could benefit from looking > > into, e.g. the usage of the worker thread - I think that may not be > > needed anymore, with XSystemShellExecute not blocking. > > > > Yes, that might work as well. Should I try this and resend the patch > or should I wait for your other suggestions as well ? > Also is this the right time to apply for the FOSDEM paper or should I > wait some more time for the patch to mature ? > Hi Suren,
so, just pushed your patches to master - added two tweaks: * FileChangedChecker moved out to svtools/source/misc/filechangedchecker.cxx * and to make it not rely on writer internals, using a generic callback mechanism from boost (the boost::function0 you see there) - that works by stuffing a little container into it, that holds the filename and SwWrtShell pointer, and just gets executed by FileChangeChecker the same as the ExternalProcessClass_Impl::CloseEvent was before. What do you think of this change? And of course great work so far, would thus be cool if you could keep improving it - maybe add it for the other applications as well? ;) Cheers, -- Thorsten
pgpIMtxwwfe1c.pgp
Description: PGP signature
_______________________________________________ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice