[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Octave-bug-tracker] [bug #58382] Incomplete symbol info when saving var
From: |
Markus Mützel |
Subject: |
[Octave-bug-tracker] [bug #58382] Incomplete symbol info when saving variables in a script |
Date: |
Sun, 17 May 2020 09:46:02 -0400 (EDT) |
User-agent: |
Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:77.0) Gecko/20100101 Firefox/77.0 |
Update of bug #58382 (project octave):
Category: GUI => Interpreter
Status: None => Confirmed
Summary: Wrong scope when saving variables with "Run
Selection" => Incomplete symbol info when saving variables in a script
_______________________________________________________
Follow-up Comment #4:
It looks like nothing needs to be done for "Run Selection" in this case.
Re-titling the bug to better reflect the underlying issue.
Comment #1 gives good instructions to reproduce.
The problem is probably with something that happens inside this function in
load-save.cc:
size_t load_save_system::save_vars (std::ostream& os,
const std::string& pattern,
const load_save_format& fmt,
bool save_as_floats)
{
tree_evaluator& tw = m_interpreter.get_evaluator ();
symbol_info_list syminfo_list = tw.glob_symbol_info (pattern);
size_t saved = 0;
for (const auto& syminfo : syminfo_list)
{
do_save (os, syminfo, fmt, save_as_floats);
saved++;
}
return saved;
}
I don't know if a script should share its symbol info with its parent, or
something different should be done here.
Adding jwe to the CC list since I think he is the only one who has a good
understanding of the symbol info implementation.
_______________________________________________________
Reply to this item at:
<https://savannah.gnu.org/bugs/?58382>
_______________________________________________
Message sent via Savannah
https://savannah.gnu.org/