diff options
author | Stephan Bergmann <sbergman@redhat.com> | 2012-06-20 16:56:22 +0200 |
---|---|---|
committer | Stephan Bergmann <sbergman@redhat.com> | 2012-06-20 18:06:38 +0200 |
commit | 796d273925c92eb91cc6ebef5caad01ccf3308b3 (patch) | |
tree | 1591553a09765e685fabb11c84765f00c18016bf /desktop/source | |
parent | removed useless InsertFunctions.hxx (diff) | |
download | core-796d273925c92eb91cc6ebef5caad01ccf3308b3.tar.gz core-796d273925c92eb91cc6ebef5caad01ccf3308b3.zip |
Clarify SIGHUP behavior
The combination of marking it as ACT_IGNORE and osl_Signal_Terminate was
strange, as osl_Signal_Terminate caused SalMainPipeExchangeSignal_impl
(desktop/source/app/officeipcthread.cxx) to call DisableOfficeIPCThread, but
ACT_IGNORE (instead of ACT_EXIT, like other signals marked osl_Signal_Terminate)
caused the process to continue running. The net effect was that calling an
additional soffice instance after a SIGHUP would no longer connect to the first
instance, but instead would lead to two soffice instances running simultaneously
on the same UserInstallation.
Marking SIGHUP as ACT_HIDE most closely resembles the old behavior (of having
~no effect). Other options would be to mark it as ACT_IGNORE and
osl_Signal_System or as ACT_EXIT and osl_Signal_Terminate, which would both
result in instant termination. I do not know if that wouldn't actually be
better.
Change-Id: Id1ee7c1108765301e6010e0e05d6c1ff5894a414
Diffstat (limited to 'desktop/source')
0 files changed, 0 insertions, 0 deletions