Opened 16 years ago

Closed 16 years ago

#705 closed Patches (Fixed)

VARNAMES bjam rule doesn't work for global module

Reported by: markovitch Owned by: René Rivera
Milestone: Component: None
Version: None Severity:
Keywords: Cc:

Description

VARNAMES bjam rule always returns empty list for the
global module. 
This is due to the fact that builtin_varnames relies
upon module_t::variables struct member being not NULL
if there are variables in the module, which is wrong
for the "currently running module" (i.e.
frame->module->variables is _always_ NULL). This is due
to the way enter_module() and var_hash_swap() work.
Since builtin_varnames() always runs in the global
module context, global module's 'variables' member is
always NULL when builtin_varnames() is called.

The proposed patch fixes this bug.

Change History (2)

comment:1 by René Rivera, 16 years ago

Logged In: YES 
user_id=33595

Applied... Thanks for the patch :-)

comment:2 by René Rivera, 16 years ago

Status: assignedclosed
Note: See TracTickets for help on using tickets.