Hi all, please your input. In the .toString() method of the IChemObject implementations (Atom, etc) the content of the IChemObject is dumped... useful to see what is happening. To be compatible with logging tools, it does so on one line, which works reasonably well, though it is quite lengthy...
Now, the latter can be addressed by not dumping to the String output all fields which are NULL. This happens more and more in trunk, so I think we should make use of that. So, instead of having this output: Isotope(0, EM:-1.0, AB:-1.0, Element(C, ID:null, AN:0)) we would dump: Isotope(0, EM:-1.0, AB:-1.0, Element(C, AN:0)) The effect would be better and better the more we move to NULL as value for UNSET instead of things like -1.0. So, my question is: does anyone have reasons to argue against removing the output of NULL fields? Egon -- ---- http://chem-bla-ics.blogspot.com/ ------------------------------------------------------------------------- This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/ _______________________________________________ Cdk-user mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/cdk-user
