From 5b5739df1cda2fef43e69de11455c17a8b1aef09 Mon Sep 17 00:00:00 2001 From: usa Date: Mon, 30 Jul 2018 14:06:52 +0000 Subject: merge revision(s) 63698: [Backport #14856] lib/drb/extservm.rb (service): do not return `false' invoke_service_command may set entries in @servers to `false', making it incompatible with the intended use of the safe navigation operator. This caused occasional DRb test failures, but they were hidden with automatic retry. [ruby-core:87524] [Bug #14856] Fixes: r53111 ("use safe navigation operator") commit 059c9c1cf371e049c7481c78b76e9620da52757f [GH-1142] git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/branches/ruby_2_4@64128 b2dd03c8-39d4-4d8f-98ff-823fe69b080e --- lib/drb/extservm.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'lib') diff --git a/lib/drb/extservm.rb b/lib/drb/extservm.rb index 7e70a3cd82..5649da5c72 100644 --- a/lib/drb/extservm.rb +++ b/lib/drb/extservm.rb @@ -38,7 +38,7 @@ module DRb synchronize do while true server = @servers[name] - return server if server&.alive? + return server if server && server.alive? # server may be `false' invoke_service(name) @cond.wait end -- cgit v1.2.3