Closed IanButterworth closed 4 years ago
Looks fine but I can't test it, I trust it works okay? Are you going to bump VERSION in this PR?
You can merge and release?
I've tested that this works in a "working" system, but I've not been able to construct a reliable error setup for catching the error state that would print within implemented(name, hNode)
, so that part is untested
But even if it doesn't catch the error, it will throw(err)
so should be ok
I'll merge & release
Yes that path looks safe
Thanks @ianshmean
Good thing you questioned that. I was missing an interpolation on the error message
Attempt to fix #65
Makes it a bit easier to figure out what's going wrong when you get a
Spinnaker SDK error: SPINNAKER_ERR_INVALID_HANDLE(-1006)