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/app | |
parent | bf729a27bdc4c5b9a3643862695d676f1324b20a (diff) |
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/app')
0 files changed, 0 insertions, 0 deletions