Mantis Bugtracker
Main | My View | View Issues | Change Log | Roadmap | Docs
  

Viewing Issue Simple Details [ Jump to Notes ] [ View Advanced ] [ Issue History ] [ Print ]
ID Category Severity Reproducibility Date Submitted Last Update
0000659 [Ecere SDK] compiler minor have not tried 2011-10-17 02:28 2012-03-29 03:34
Reporter jerome View Status public  
Assigned To
Priority low Resolution open  
Status new   Product Version
Summary 0000659: Method/Symbol mixups on reverse-ordered Window hierarchy (only design issues remain) (ide/libec)
Description A Symbol object is created and assigned to a Method object in firstPass.ec, ~line 462 -- ProcessClass().

That Method object eventually would be deleted (if not for the patch mentionned below) when the hierarchy is fixed and the system realizes it's a virtual method, but there's no way to go back and make the function declarator's symbol point to the new method.

A proper fix would be to ensure hierarchies are established in firstPass.ec before assigning Method objects to AST elements (Split in 2 passes, first do all the eSystem_RegisterClass, then do the methods).
Additional Information
Tags No tags attached.
Attached Files ? file icon GlobalSettingsDialog.ec [^] (35,173 bytes) 2011-10-17 02:28

- Relationships

-  Notes
(0000481)
jerome (administrator)
2011-10-17 03:43

 In firstPass, a eSystem_RegisterClass is freeing up the method that was saved into a function declarator Symbol beforehand.
(0000482)
jerome (administrator)
2011-10-17 18:17
edited on: 2011-10-18 13:10

 Added a temporary patch to prevent a crash:

https://github.com/ecere/sdk/commit/84d925c3301f3a5ecb77146c11bd92060f46eb03 [^]

The virtual method DOES get called.

But the auto-completion fails on event methods
(0000484)
jerome (administrator)
2011-10-18 13:42

 Fixed auto completion issue in https://github.com/ecere/sdk/commit/dc1d794eb67fa8fc29e69953b8a6277423881574 [^]

Only leak remains.
(0000485)
jerome (administrator)
2011-10-18 14:10

 The leak wasn't actually there since crash patch, until auto-completion was put.
Worked around the leak in https://github.com/ecere/sdk/commit/5998fb97b79a3f610963854bc78d271186d0951e [^]

- Issue History
Date Modified Username Field Change
2011-10-17 02:28 jerome New Issue
2011-10-17 02:28 jerome File Added: GlobalSettingsDialog.ec
2011-10-17 03:43 jerome Note Added: 0000481
2011-10-17 18:17 jerome Note Added: 0000482
2011-10-18 13:09 jerome Priority immediate => urgent
2011-10-18 13:09 jerome Target Version 0.44 pre-release 2 => 0.44 pre-release 3
2011-10-18 13:10 jerome Note Edited: 0000482
2011-10-18 13:10 jerome Note Added: 0000483
2011-10-18 13:11 jerome Severity crash => minor
2011-10-18 13:11 jerome Summary ide/autocomplete: Crash on 'void OnCreate(' preceding another void Load() method in CompilerDirectoriesTab => ide/autocomplete: Leak and Auto-Completion failure on reverse-ordered Window hierarchy
2011-10-18 13:42 jerome Note Added: 0000484
2011-10-18 13:42 jerome Summary ide/autocomplete: Leak and Auto-Completion failure on reverse-ordered Window hierarchy => ide/libec: Leak on reverse-ordered Window hierarchy
2011-10-18 13:44 jerome Note Edited: 0000483
2011-10-18 13:44 jerome Note Deleted: 0000483
2011-10-18 13:50 jerome Description Updated
2011-10-18 14:10 jerome Note Added: 0000485
2011-10-18 14:11 jerome Priority urgent => low
2011-10-18 14:11 jerome Target Version 0.44 pre-release 3 => 0.45
2011-10-18 14:11 jerome Summary ide/libec: Leak on reverse-ordered Window hierarchy => ide/libec: Method/Symbol mixups on reverse-ordered Window hierarchy (only design issues remain)
2012-03-08 10:02 redj Target Version old 0.45 => 0.45 Ginkakuji
2012-03-29 02:52 redj Category => ide
2012-03-29 02:52 redj Project @3@ => Ecere SDK
2012-03-29 03:34 redj Category ide => ec
2012-03-29 03:34 redj Summary ide/libec: Method/Symbol mixups on reverse-ordered Window hierarchy (only design issues remain) => Method/Symbol mixups on reverse-ordered Window hierarchy (only design issues remain) (ide/libec)
2012-03-29 03:34 redj Category ec => compiler

Mantis 1.1.1[^]
Copyright © 2000 - 2008 Mantis Group
Powered by Mantis Bugtracker