octave-patch-tracker
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Octave-patch-tracker] [patch #7888] add "remove" to autoload


From: Juan Pablo Carbajal
Subject: [Octave-patch-tracker] [patch #7888] add "remove" to autoload
Date: Mon, 19 Nov 2012 14:51:28 +0000
User-agent: Mozilla/5.0 (X11; Linux i686) AppleWebKit/537.11 (KHTML, like Gecko) Chrome/23.0.1271.64 Safari/537.11

URL:
  <http://savannah.gnu.org/patch/?7888>

                 Summary: add "remove" to autoload
                 Project: GNU Octave
            Submitted by: juanpi
            Submitted on: Mon 19 Nov 2012 02:51:27 PM GMT
                Category: None
                Priority: 5 - Normal
                  Status: None
                 Privacy: Public
             Assigned to: None
        Originator Email: 
             Open/Closed: Open
         Discussion Lock: Any

    _______________________________________________________

Details:

Hi,
Realted thread: 
http://octave.1599824.n4.nabble.com/reverse-of-autoload-td4644524.html

Please find attached a change set that adds the option "remove" to autoload.

The function is extended to accept a third optional argument "remove"
that causes the function to be cleared and removed from autoload_map

The second argument is not used except for the checks (which are
unnecessary in the case of "remove"). So, I can redo the changes to
accept
autoload("foo","remove")

With the current check "remove" is not confused with a absolute path,
so it should produce no clash. Please, let me know.

Also, if "foo" is not in the autoload_map nothing is printed nor
returned. Shall I give a warning?

I did not know if symbol_table::clear_function removes the function
from memory thence there is a comment witha question mark. If that is
the right way of removing a function from memory please remove the
question mark.


Thanks

JPi



    _______________________________________________________

File Attachments:


-------------------------------------------------------
Date: Mon 19 Nov 2012 02:51:27 PM GMT  Name: changeset_autoload  Size: 2kB  
By: juanpi

<http://savannah.gnu.org/patch/download.php?file_id=26932>

    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/patch/?7888>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.gnu.org/




reply via email to

[Prev in Thread] Current Thread [Next in Thread]