all: make sure Context.Close is called

Co-authored-by: Drew O'Meara <[email protected]>

Author: drew-512

examples/embedding/main.go

@@ -27,6 +27,9 @@ func runWithFile(pyFile string) error {
 
 	// See type Context interface and related docs
 	ctx := py.NewContext(py.DefaultContextOpts())
+	
+	// This drives modules being able to perform cleanup and release resources 
+	defer ctx.Close()
 
 	var err error
 	if len(pyFile) == 0 {

examples/embedding/mylib.module.go

@@ -47,6 +47,9 @@ func init() {
 			"GO_VERSION": py.String(fmt.Sprintf("%s on %s %s", runtime.Version(), runtime.GOOS, runtime.GOARCH)),
 			"MYLIB_VERS": py.String("Vacation 1.0 by Fletch F. Fletcher"),
 		},
+		OnContextClosed: func(instance *py.Module) {
+			fmt.Print("<<< host py.Context of py.Module instance closing >>>\n+++\n")
+		},
 	})
 }
 

examples/embedding/testdata/embedding_out_golden.txt

@@ -15,3 +15,5 @@ Spring Break itinerary:
 
 I bet Monaco will be the best!
 
+<<< host py.Context of py.Module instance closing >>>
++++

examples/multi-context/main.go

@@ -128,6 +128,9 @@ func RunMultiPi(numWorkers, numTimes int) time.Duration {
 				w.ctx.RunCode(jobCode, w.main.Globals, w.main.Globals, nil)
 			}
 			workersRunning.Done()
+
+			// This drives modules being able to perform cleanup and release resources 
+			w.ctx.Close()
 		}()
 	}
 

main.go

@@ -48,6 +48,7 @@ func xmain(args []string) {
 	opts := py.DefaultContextOpts()
 	opts.SysArgs = args
 	ctx := py.NewContext(opts)
+	defer ctx.Close()
 
 	if *cpuprofile != "" {
 		f, err := os.Create(*cpuprofile)