[Octave-bug-tracker] [bug #66727] [octave-forge] (windows) Loading/unloading windows package leaves variables in workspace

[Philip Nienhuis]

Follow-up Comment #3, bug #66727 (group octave):

The lines removing the variables are missing; in the patch e.g.
__load_windows__ () :

 :
 :
+function __load_windows__ ()
+  # on package load, attempt to load docs
+  try
+    pkg_dir = fileparts (fullfile (mfilename ("fullpath")));
+    doc_file = fullfile (pkg_dir, "doc", "windows.qch");
+    doc_file = strrep (doc_file, '\', '/');
+    if exist(doc_file, "file")
+      if exist("__event_manager_register_documentation__")
+        __event_manager_register_documentation__ (doc_file);
+      elseif exist("__event_manager_register_doc__")
+        __event_manager_register_doc__ (doc_file);
+      endif
+    endif
+  catch
+    # do nothing
+  end_try_catch
+endfunction

inst/__unload_windows__.m
Diff
Switch to side-by-side view

--- a
+++ b/inst/__unload_windows__.m
@@ -0,0 +1,40 @@
 :
 :

they should be after the end_try_catch.

BTW how are these new functions invoked? (just curious) in the windows package
I couldn't find a function invoking them.
E.g., in the io package PKG_ADD just contains a single line calling
__init_io__.m , similarly in PKG_DEL.


    _______________________________________________________

Reply to this item at:

  <https://savannah.gnu.org/bugs/?66727>

_______________________________________________
Message sent via Savannah
https://savannah.gnu.org/

signature.asc
Description: PGP signature