This is an automated email from the git hooks/post-receive script. guillem pushed a commit to branch main in repository dpkg.
View the commit online: https://git.dpkg.org/cgit/dpkg/dpkg.git/commit/?id=0c8898235cd37301569daa51abace7f3aac2a7d1 commit 0c8898235cd37301569daa51abace7f3aac2a7d1 Author: Guillem Jover <[email protected]> AuthorDate: Mon Nov 7 22:51:57 2022 +0100 Dpkg: Rename _sanity_check and _sanity_check_opts to _check_opts Make it clear we are checking options, and remove the unnecessary sanity term. --- scripts/Dpkg/Compression/Process.pm | 6 +++--- scripts/Dpkg/IPC.pm | 4 ++-- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/scripts/Dpkg/Compression/Process.pm b/scripts/Dpkg/Compression/Process.pm index 9b733cc74..34d99bd70 100644 --- a/scripts/Dpkg/Compression/Process.pm +++ b/scripts/Dpkg/Compression/Process.pm @@ -118,7 +118,7 @@ sub get_uncompress_cmdline { return (@{compression_get_property($self->{compression}, 'decomp_prog')}); } -sub _sanity_check { +sub _check_opts { my ($self, %opts) = @_; # Check for proper cleaning before new start error(g_('Dpkg::Compression::Process can only start one subprocess at a time')) @@ -149,7 +149,7 @@ properly close the sub-process (and verify that it exited without error). sub compress { my ($self, %opts) = @_; - $self->_sanity_check(%opts); + $self->_check_opts(%opts); my @prog = $self->get_compress_cmdline(); $opts{exec} = \@prog; $self->{cmdline} = "@prog"; @@ -172,7 +172,7 @@ properly close the sub-process (and verify that it exited without error). sub uncompress { my ($self, %opts) = @_; - $self->_sanity_check(%opts); + $self->_check_opts(%opts); my @prog = $self->get_uncompress_cmdline(); $opts{exec} = \@prog; $self->{cmdline} = "@prog"; diff --git a/scripts/Dpkg/IPC.pm b/scripts/Dpkg/IPC.pm index 52c05d0b3..250364203 100644 --- a/scripts/Dpkg/IPC.pm +++ b/scripts/Dpkg/IPC.pm @@ -145,7 +145,7 @@ array to their default dispositions before calling exec. =cut -sub _sanity_check_opts { +sub _check_opts { my (%opts) = @_; croak 'exec parameter is mandatory in spawn()' @@ -208,7 +208,7 @@ sub spawn { my (%opts) = @_; my @prog; - _sanity_check_opts(%opts); + _check_opts(%opts); $opts{close_in_child} //= []; if (ref($opts{exec}) =~ /ARRAY/) { push @prog, @{$opts{exec}}; -- Dpkg.Org's dpkg
