Don't pass null String to LayoutManager#addLayoutComponent(String, Component)

Mark Wielaard mark@klomp.org
Sun Jan 22 20:14:00 GMT 2006


Hi James,

On Thu, 2006-01-19 at 11:42 -0500, James Damour wrote:
> Beware!!  I don't have a working mauve test suite so this patch is 
> completely untested!!!
>
> 2006-01-19  James Damour  <James.Damour@corp.request.com>
> 
> 	* java/awt/Container/addImpl.java
> 	(test): Added check for non-null String passed to addLayoutComponent.

Thanks, checked in as attached.
I'll also commit a fix to GNU Classpath for this.

BTW you can just run one test as follows:
jikes gnu/testlet/*.java gnu/testlet/java/awt/Container/addImpl.java
echo gnu.testlet.java.awt.Container.addImpl | jamvm gnu.testlet.SimpleTestHarness

cheers,

Mark
-------------- next part --------------
A non-text attachment was scrubbed...
Name: addImpl.patch
Type: text/x-patch
Size: 1581 bytes
Desc: not available
URL: <http://sourceware.org/pipermail/mauve-patches/attachments/20060122/8a1c61c7/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part
URL: <http://sourceware.org/pipermail/mauve-patches/attachments/20060122/8a1c61c7/attachment.sig>


More information about the Mauve-patches mailing list