[jdom-interest] Redundant Indent methods in XMLOutputter

Elliotte Rusty Harold elharo at metalab.unc.edu
Tue Apr 9 14:07:54 PDT 2002


There are four separate indent methods in XMLOutputter, none of which 
are marked as deprecated:


public void *setIndent*(java.lang.String indent)
public void *setIndent*(boolean doIndent)
public void *setIndent*(int size)
public void *setIndentSize*(int indentSize)


The last two methods appear to do exactly the same thing.  I suggest we 
deprecate and eventually delete one of these two methods.

I also suggest that we either remove * setIndent*(boolean doIndent) or 
that we replace the first two methods with two separate methods like this:

setIndentString(String indent)
setDoIndent(boolean true)

Right now it's quite confusing to figure wout whether or not you've 
actually uscceeded in turning on indenting in the event of a conflict 
between the setIndent methods.

-- 
+-----------------------+------------------------+-------------------+
| Elliotte Rusty Harold | elharo at metalab.unc.edu | Writer/Programmer |
+-----------------------+------------------------+-------------------+ 
|           The XML Bible, 2nd Edition (IDG Books, 2001)             |
|             http://www.cafeconleche.org/books/bible2/              |
|   http://www.amazon.com/exec/obidos/ISBN=0764547607/cafeaulaitA/   |
+----------------------------------+---------------------------------+
|  Read Cafe au Lait for Java News:   http://www.cafeaulait.org/     | 
|  Read Cafe con Leche for XML News:  http://www.cafeconleche.org/   |
+----------------------------------+---------------------------------+






More information about the jdom-interest mailing list