Bugs item #1330309, was opened at 2005-10-18 16:07 Message generated for change (Comment added) made by sf-robot You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=482468&aid=1330309&group_id=56967
Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: None Group: Pathfinder CVS Head >Status: Closed Resolution: Fixed Priority: 5 Private: No Submitted By: Peter Boncz (boncz) Assigned to: Jens Teubner (teubner) Summary: module bug Initial Comment: Hi Jens, Thanks a lot for implementing those modules. However, the following issues: - it seems as 'declare variable' that put variables into the static context are still ignored - the bug1 script hits a core dump. it attempts to load two modules and calls a function from the second modules. That function in turn, is defined in the first module. - the bug2 script omits the import for the second module. Here it just says 'error loading in module ..'. I think W3C semantics is that indeed an error should be given (because the query needs to explicitly load all modules it uses). However, the error message could be improved. best, Peter ---------------------------------------------------------------------- >Comment By: SourceForge Robot (sf-robot) Date: 2007-12-20 19:20 Message: Logged In: YES user_id=1312539 Originator: NO This Tracker item was closed automatically by the system. It was previously set to a Pending status, and the original submitter did not respond within 365 days (the time period specified by the administrator of this Tracker). ---------------------------------------------------------------------- Comment By: SourceForge Robot (sf-robot) Date: 2006-12-19 19:20 Message: Logged In: YES user_id=1312539 Originator: NO This Tracker item was closed automatically by the system. It was previously set to a Pending status, and the original submitter did not respond within 365 days (the time period specified by the administrator of this Tracker). ---------------------------------------------------------------------- Comment By: Stefan Manegold (stmane) Date: 2005-12-19 13:38 Message: Logged In: YES user_id=572415 BugDay_2005-12-19, stmane: NO TEST / POSTPONED: need more time to turn the attached scripts into a proper test... ---------------------------------------------------------------------- Comment By: Jens Teubner (teubner) Date: 2005-10-28 02:58 Message: Logged In: YES user_id=731390 Hi Peter, I've just committed a fix for fs.brg that hopefully fixes the core dump that you mentioned. Can you test? Jens ---------------------------------------------------------------------- Comment By: Jens Teubner (teubner) Date: 2005-10-23 11:06 Message: Logged In: YES user_id=731390 Hi Peter, thanks for the report. (1) I've just committed some changes regarding variables declared in modules. (If a module was given as Pathfinder's input an that module loads some other module, it was handled incorrectly.) However, Pathfinder's behavior was partially intended (but we may change that to suit our needs). I had implemented module importing via a work list; modules are added to that list upon `import module' statements, and loaded until the list is processed. This way, we don't really have the information about which module has imported which other module, but rather treat them as they were all loaded independently from the main query. In that case, variables are not visible between modules. This "invisibility" in a sense is a feature, though Pathfinder handles it too strict. But maybe it is better to drop that restriction and be too lax instead. I will change that. (2) The segfault produced by bug1.xq is actually a bit tricky. Function declarations and their uses have to be translated in correct order. Parse trees with modules imported look a bit more tricky than they looked before, and I'll have to see how I can still handle that correctly with Burg. Stay tuned. (3) I actually cannot reproduce that one right now. The error actually indicates there was some problem retrieving the file. Due to the above mentioned way of loading modules, Pathfinder cannot be intelligent enough to trigger the error that you suspected. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=482468&aid=1330309&group_id=56967 ------------------------------------------------------------------------- This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2005. http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/ _______________________________________________ Monetdb-bugs mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/monetdb-bugs
