View Issue Details

IDProjectCategoryView StatusLast Update
0000688Ecere SDKecerepublic2016-07-12 03:06
Reporterredj Assigned Toredj  
PrioritynormalSeverityminorReproducibilityhave not tried
Status assignedResolutionopen 
PlatformCross-PlatformOS\o/OS Version;-)
Target Version0.44.16 
Summary0000688: api review: global settings with json
Descriptionreview how this is used in the IDE
IDESettingsContainer settings
{
   driver = "JSON";
   data = ideSettings;
   dataOwner = &ideSettings;
};
TagsNo tags attached.

Activities

redj

2012-03-08 16:39

administrator   ~0000516

this is to be reviewed... please file a mantis issue
I know it has been.
IDESettings ideSettings { }; // instantiate the IDESettings class from the IDESettings.ec file. Do this at a global level so that all methods can access settings.

but why we need to do all this
instead of IDESettings settings { };
"JSON" should be default
this data = ideSettings / dataOwner = & is nonsense
please put my critic in the mantis issue

we need to do all this because JSON is not default (keeping in line with your principles, don't break code that depends on ecere)...

all commented out code ;|
why is it not default?
ah

Issue History

Date Modified Username Field Change
2012-03-08 16:34 redj New Issue
2012-03-08 16:39 redj Note Added: 0000516
2012-03-08 16:48 redj Target Version 0.46.1 => 0.45 Ginkakuji
2012-03-29 07:53 redj Category => Ecere Runtime Library
2012-03-29 07:53 redj Project @1@ => Ecere SDK
2014-07-11 18:46 jerome Target Version 0.45 Ginkakuji => 0.46.1
2016-04-19 05:19 jerome Target Version 0.46.1 => 0.46 eC II
2016-07-12 03:06 jerome Assigned To => redj
2016-07-12 03:06 jerome Status new => assigned
2016-07-12 03:06 jerome Target Version 0.46 eC II => 0.44.16