I see this is not going to work, since isEmpty() defines itself as where one component's max (maxX, maxY, maxZ) is less than the corresponding min. So we make sense, at least, out of -1 (although as far as the implementation is concerned, any negative value works just as well).
On Oct 1, 2013, at 3:13 PM, Richard Bair <[email protected]> wrote: > Hi, > > I'm looking at https://javafx-jira.kenai.com/browse/RT-23446, where the > argument is made that the width / height of a node (specifically, a Region's > prefWidth, minWidth, maxWidth, prefHeight, minHeight, maxHeight) should never > be negative. While looking at this, I noticed that in Node, the prefWidth > method relies on the layoutBounds.getWidth(). However, the Bounds class > itself does not appear to do any validation of the parameters passed to the > Bounds. There are no checks for NaN, and no checks for negative width, > height, depth. > > Is there any reason why we should allow NaN, or negative width / height / > depth for Bounds? > > Richard
