diff options
author | Marius Kintel <marius@kintel.net> | 2012-01-04 17:05:30 (GMT) |
---|---|---|
committer | Marius Kintel <marius@kintel.net> | 2012-01-04 17:05:30 (GMT) |
commit | 9856607d278d3843d56dab8325a552351accd2cf (patch) | |
tree | 151074d2394a0929737292fa10d0a42cf8cf8fc2 /src | |
parent | 9bac643ed815d18af07dbdd78f68d54e34ab68b9 (diff) | |
parent | 9f16204889c8e42785bf82d6a5db74c80e0027b6 (diff) |
Merge pull request #54 from openscad/parserfix
fix crash bug rpt by nop head - 'use <x.scad>' when x.scad doesnt exist
Diffstat (limited to 'src')
-rw-r--r-- | src/parser.y | 15 |
1 files changed, 9 insertions, 6 deletions
diff --git a/src/parser.y b/src/parser.y index b0df50d..3dd933c 100644 --- a/src/parser.y +++ b/src/parser.y @@ -588,12 +588,15 @@ AbstractModule *parse(const char *text, const char *path, int debug) if (!module) return NULL; - BOOST_FOREACH(Module::ModuleContainer::value_type &m, module->usedlibs) { - module->usedlibs[m.first] = Module::compile_library(m.first); - if (!module->usedlibs[m.first]) { - PRINTF("WARNING: Failed to compile library `%s'.", m.first.c_str()); - module->usedlibs.erase(m.first); - } + // Iterating manually since we want to modify the container while iterating + Module::ModuleContainer::iterator iter = module->usedlibs.begin(); + while (iter != module->usedlibs.end()) { + Module::ModuleContainer::iterator curr = iter++; + curr->second = Module::compile_library(curr->first); + if (!curr->second) { + PRINTF("WARNING: Failed to compile library `%s'.", curr->first.c_str()); + module->usedlibs.erase(curr); + } } parser_error_pos = -1; |