How to add Platform options ? about reko HOT 5 CLOSED

I'm not too keen on popping up user interfaces in the user's face. Rather, in the project browser, let's consider adding a new treenode, a child of the executable, which would be labelled AmigaOS or what have you. Each platform can then have its own custom user interface which could accept user data and persist it in the *.dcproj file.

I can mock this up in the "gui-development" branch over the weekend, so you can take a look.

from reko.

Revision 76ad42d of the "gui-development" branch now shows the Platform in the project browser. If the platform is AmigaOS, the Platform will be clickable and show some UI, You can add controls on the AmigaOSProperties user control and react to changes by saving them to the instance of AmigaOSPlatform in the AmigaOSPropertiesInteractor class.

@nemerle , @halsten , open up a file from ~/subjects/Hunk-m68k and let me know what you think.

from reko.

@nemerle: you will need to add the requisite properties to the AmigaOSPlatform and MsdosPlatform to store all these hardware configuration details. Then, the loading and saving of projects has to be modified to ask each platform to save / load platform specific options to/from an XmlTextWriter / XmlTextReader. The contents of this Xml will then be added to the .dcconfig file as an opaque blob.

I need some time to investigate how to save XML blobs to a file using XmlSerializer in an elegant way, In the meantime, you can work on the user interface I presented you, adding textboxes, comboboxes etc. to your heart's content

from reko.

@nemerle: commit ce1d57f added the virtual methods LoadUserOptions and SaveUserOptions to the Platform class. You're welcome to extend the AmigaOSPlatform class by overriding those methods and implementing the appropriate functionality. The AmigaOSPlatformDesigner user interface should save user changes into a Dictionary<string,object> which can be kept in the AmigaOSPlatform class.

from reko.

Both architectures and platforms have supported options for a while now.

from reko.