Hi James,

It might be cleaner to make null an illegal value for the addText() call, and throw an IllegalArgumentException (or other exception) for it. Silently ignoring the call might create problems down the line ("I make three addText() calls, but there are only two text nodes in the result - what's going on?").

  - Dennis

James Strachan wrote:

 Element element = ...;element.addText ( null ); If this is the case then maybe we should explicitly check for null strings and quitely ignore them... 
James
_______________________________________________ dom4j-dev mailing list [EMAIL PROTECTED] https://lists.sourceforge.net/lists/listinfo/dom4j-dev

Reply via email to