Fix reference counts in nrnexec.

[1uc]

The function nrnexec is wrapped and then registered via PyMethodDef. Therefore is must satisfy the requirements: https://docs.python.org/3.12/c-api/structures.html#c.PyCFunction

Meaning it must return a new reference. Hence, it must the Py_RETURN_{TRUE,FALSE} macros.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[JCGoran]

Could we use https://docs.python.org/3/c-api/bool.html#c.PyBool_FromLong instead? It's part of the stable ABI, and returns a new reference.
For context, I'd prefer to use functionality from the stable ABI (only if possible, and if the performance penalty is minimal) for any new changes, as it makes it easier to (eventually, maybe) support the stable ABI with NEURON.

[bbpbuildbot]

Logfiles from GitLab pipeline #239859 (:no_entry:) have been uploaded here!

Status and direct links:

• ✅ mac_m1_cmake_build: [cmake, ON, OFF, OFF, address]
• ✅ spack_setup
• ✅ build:nmodl
• ✅ build:neuron:nmodl:intel:legacy
• ✅ build:neuron:nmodl:intel:shared
• ✅ build:neuron:nmodl:nvhpc:acc:legacy
• ✅ build:neuron:nmodl:nvhpc:acc:shared
• ✅ build:neuron:nmodl:nvhpc:omp:legacy
• ✅ build:neuron:nmodl:nvhpc:omp
• ✅ test:neuron:nmodl:intel:legacy
• ✅ test:neuron:nmodl:intel:shared
• ⛔ test:neuron:nmodl:nvhpc:acc:legacy
• ⛔ test:neuron:nmodl:nvhpc:acc:shared
• ⛔ test:neuron:nmodl:nvhpc:omp:legacy
• ⛔ test:neuron:nmodl:nvhpc:omp

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 67.07%. Comparing base (d177f5b) to head (64e5eaf).
Report is 1 commits behind head on master.

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #3269   +/-   ##
=======================================
  Coverage   67.07%   67.07%           
=======================================
  Files         571      571           
  Lines      111110   111111    +1     
=======================================
+ Hits        74528    74529    +1     
  Misses      36582    36582           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[azure-pipelines]

✔️ 64e5eaf -> Azure artifacts URL