I would like to suggest some changes to the Parameter and Message base
classes in LLRPDataType.cs

Firstly I would like to propose that they both be made abstract classes as
it doesn't make sense to be able to instantiate either.  This would also
allow us to define method signature that inheriting classes would have to
implement (similar to how interfaces work).  

Secondly I would like to propose adding a new method that utilizes the
abstractness of the class:
public abstract void LoadXmlNode(XmlNode node);

This would guarantee every inheriting class has a method to load data into
itself from an XmlNode.

I have found this to be extremely useful in my testing so if you have any
questions or doubts please let me know.  If it appears there is a positive
consensus on this I can go ahead and submit a patch to make these changes.

-Matt



-------------------------------------------------------------------------
SF.Net email is sponsored by: 
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://sourceforge.net/services/buy/index.php
_______________________________________________
llrp-toolkit-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/llrp-toolkit-devel

Reply via email to