Tidy

Author: Saul Cooperman

src/pystack/_pystack/interpreter.cpp

@@ -17,7 +17,7 @@ InterpreterUtils::getNextInterpreter(
     return is.getField(&py_is_v::o_next);
 }
 
-int
+int64_t
 InterpreterUtils::getInterpreterId(
         const std::shared_ptr<const AbstractProcessManager>& manager,
         remote_addr_t interpreter_addr)

src/pystack/_pystack/interpreter.h

@@ -16,7 +16,7 @@ class InterpreterUtils
             const std::shared_ptr<const AbstractProcessManager>& manager,
             remote_addr_t interpreter_addr);
 
-    static int getInterpreterId(
+    static int64_t getInterpreterId(
             const std::shared_ptr<const AbstractProcessManager>& manager,
             remote_addr_t interpreter_addr);
 };

src/pystack/_pystack/interpreter.pxd

@@ -1,12 +1,13 @@
 from _pystack.mem cimport remote_addr_t
 from _pystack.process cimport AbstractProcessManager
+from libc.stdint cimport int64_t
 from libcpp.memory cimport shared_ptr
 
 
 cdef extern from "interpreter.h" namespace "pystack":
     cdef cppclass  InterpreterUtils:
         @staticmethod
-        remote_addr_t getNextInterpreter(shared_ptr[AbstractProcessManager] manager, remote_addr_t interpreter_addr) except+
+        remote_addr_t getNextInterpreter(shared_ptr[AbstractProcessManager] manager, remote_addr_t interpreter_addr) except +
 
         @staticmethod
-        int getInterpreterId(shared_ptr[AbstractProcessManager] manager, remote_addr_t interpreter_addr) except+
+        int64_t getInterpreterId(shared_ptr[AbstractProcessManager] manager, remote_addr_t interpreter_addr) except +

tests/unit/test_traceback_formatter.py

@@ -1709,7 +1709,7 @@ def test_print_thread_with_subinterpreters(capsys):
 
     # THEN
     captured = capsys.readouterr()
-    assert "Interpreter-Unknown (main)" in captured.out
+    assert "Interpreter-0 (main)" in captured.out
     # Lines should be indented with 2 spaces
     assert "  line1\n" in captured.out
     assert "  line2\n" in captured.out