Is it necessary to expunge the connection here? about jarvis

[arcfide]

As mentioned in the pull request, if I understand this correctly, I think it would be better to expunge on the outside, where the object is created.

[arcfide]

I notice now that making a response includes expunging the object as a part of its work. This makes me wonder now whether it is even necessary to expunge the object when we return from HandleRequest?

[arcfide]

I'm referring to this apparently excessive expunge:

Jarvis/Source/Jarvis.dyalog

Line 446 in ccd05ff

_taskThreads←⎕TNUMS∩_taskThreads,(Connections⍎obj){t←⍺ HandleRequest ⍵ ⋄ ⎕EX t/⍕⍺}&wres

[arcfide]

d2c5e53 removes the extra expunge here, and now the only expunges are on the outside caller to HandleRequest and inside of the Respond function. Are they both necessary?

[bpbecker]

Yeah, the duplicate expunge was a remnant of cope that was lifted from elsewhere. While it's harmless I'll address it.