Project

General

Profile

Misc #12439

[PATCH cleanup] process.c (disable_child_handler_fork_child): simplify

Added by normalperson (Eric Wong) over 4 years ago. Updated about 3 years ago.

Status:
Closed
Priority:
Normal
[ruby-core:75781]

Description

process.c (disable_child_handler_fork_child): simplify

signal(2) is portable for SIG_DFL and SIG_IGN, so we do not
need the extra code for sigaction(2).

Also, execve will reset all signal handlers to default
anyways, so there is little sense in preserving old
signal handler besides SIG_IGN.

Hopefully this makes the code easier-to-understand and
maintain.

Comments? Will commit in a week or two if no response.
Thanks.


Files

#1

Updated by shyouhei (Shyouhei Urabe) over 3 years ago

  • Status changed from Open to Assigned

Updated by akr (Akira Tanaka) about 3 years ago

  • Status changed from Assigned to Closed

It seems already committed.

Also available in: Atom PDF