Use disable_sigint instead of sigatomic_(begin|end)

Author: tkf

src/PyCall.jl

@@ -24,13 +24,6 @@ if isdefined(Base, :hasproperty) # Julia 1.2
     import Base: hasproperty
 end
 
-# Python C API is not interrupt-safe.  In principle, we should
-# use sigatomic for every ccall to the Python library, but this
-# should really be fixed in Julia (#2622).  However, we will
-# use the sigatomic_begin/end functions to protect pycall and
-# similar long-running (or potentially long-running) code.
-import Base: sigatomic_begin, sigatomic_end
-
 import Conda
 import MacroTools   # because of issue #270
 import Base.Iterators: filter

src/pyeval.jl

@@ -31,16 +31,13 @@ pynamespace(m::Module) =
 # and a current "file name" to use for stack traces
 function pyeval_(s::AbstractString, globals=pynamespace(Main), locals=pynamespace(Main),
                  input_type=Py_eval_input, fname="PyCall")
-    sigatomic_begin()
-    try
+    disable_sigint() do
         o = PyObject(@pycheckn ccall((@pysym :Py_CompileString), PyPtr,
                                      (Cstring, Cstring, Cint),
                                      s, fname, input_type))
         return PyObject(@pycheckn ccall((@pysym :PyEval_EvalCode),
                                          PyPtr, (PyPtr, PyPtr, PyPtr),
                                          o, globals, locals))
-    finally
-        sigatomic_end()
     end
 end
 

src/pyfncall.jl

@@ -39,14 +39,11 @@ Sets `ret.o` to the result of the call, and returns `ret::PyObject`.
 """
 function __pycall!(ret::PyObject, pyargsptr::PyPtr, o::Union{PyObject,PyPtr},
   kw::Union{Ptr{Cvoid}, PyObject})
-    sigatomic_begin()
-    try
+    disable_sigint() do
         retptr = @pycheckn ccall((@pysym :PyObject_Call), PyPtr, (PyPtr,PyPtr,PyPtr), o,
                         pyargsptr, kw)
         pydecref_(ret)
         setfield!(ret, :o, retptr)
-    finally
-        sigatomic_end()
     end
     return ret #::PyObject
 end

src/pyiterator.jl

@@ -5,14 +5,11 @@
 
 Base.IteratorSize(::Type{PyObject}) = Base.SizeUnknown()
 function _start(po::PyObject)
-    sigatomic_begin()
-    try
+    disable_sigint() do
         o = PyObject(@pycheckn ccall((@pysym :PyObject_GetIter), PyPtr, (PyPtr,), po))
         nxt = PyObject(@pycheck ccall((@pysym :PyIter_Next), PyPtr, (PyPtr,), o))
 
         return (nxt,o)
-    finally
-        sigatomic_end()
     end
 end
 
@@ -76,12 +73,9 @@ _start(piter::PyIterator) = _start(piter.o)
 
 function Base.iterate(piter::PyIterator{T}, s=_start(piter)) where {T}
     ispynull(s[1]) && return nothing
-    sigatomic_begin()
-    try
+    disable_sigint() do
         nxt = PyObject(@pycheck ccall((@pysym :PyIter_Next), PyPtr, (PyPtr,), s[2]))
         return (convert(T,s[1]), (nxt, s[2]))
-    finally
-        sigatomic_end()
     end
 end
 function Base.iterate(po::PyObject, s=_start(po))