NullPointerException in BorderLayout when Layout Data is null about eclipse.platform.swt HOT 8 CLOSED

Similarly with null Layout Data and layout.type = SWT.VERTICAL; BorderLayout.getMax() also throws NPE as below:
Exception in thread "main" java.lang.NullPointerException
at org.eclipse.swt.layout.BorderLayout.lambda$5(BorderLayout.java:244)
at java.base/java.util.stream.ReferencePipeline$4$1.accept(ReferencePipeline.java:212)
at java.base/java.util.ArrayList$ArrayListSpliterator.forEachRemaining(ArrayList.java:1654)
at java.base/java.util.stream.AbstractPipeline.copyInto(AbstractPipeline.java:484)
at java.base/java.util.stream.AbstractPipeline.wrapAndCopyInto(AbstractPipeline.java:474)
at java.base/java.util.stream.ReduceOps$ReduceOp.evaluateSequential(ReduceOps.java:913)
at java.base/java.util.stream.AbstractPipeline.evaluate(AbstractPipeline.java:234)
at java.base/java.util.stream.IntPipeline.reduce(IntPipeline.java:496)
at java.base/java.util.stream.IntPipeline.max(IntPipeline.java:459)
at org.eclipse.swt.layout.BorderLayout.getMax(BorderLayout.java:244)
at org.eclipse.swt.layout.BorderLayout.getMax(BorderLayout.java:240)
at org.eclipse.swt.layout.BorderLayout.computeSize(BorderLayout.java:149)
at org.eclipse.swt.widgets.Composite.computeSizeInPixels(Composite.java:217)
at org.eclipse.swt.widgets.Control.computeSize(Control.java:663)
at org.eclipse.swt.widgets.Control.pack(Control.java:2198)
at org.eclipse.swt.widgets.Control.pack(Control.java:2168)
at org.eclipse.swt.snippets.Snippet379.main(Snippet379.java:50)

from eclipse.platform.swt.

I can take a look at this issue and begin working on a fix. Just want to confirm, @laeubi do you have a patch in progress already?

from eclipse.platform.swt.

@joel-majano I haven't yet prepared a patch and contributions are always welcome, so please go ahead
Just add me as a reviewer then, otherwise I'll be looking into this next week. I think the first step would be to enhance the test-case to reproduce this issue.

from eclipse.platform.swt.

I was able to recreate the NPE that Niraj commented on, but to recreate the original issue I have to set SWT.HORIZONTAL and call shell.pack() explicitly (in both cases). Are these the expected failure conditions?

from eclipse.platform.swt.

At least the stacktrace indicates that there is a pack() involved:

at org.eclipse.swt.widgets.Control.pack(Control.java:2198)

from eclipse.platform.swt.

Okay, in that case, I get this issue even if LayoutData is not null. Just taking Snippet 379, adding a shell.pack() call right before shell.open() creates the NPE. SWT.VERTICAL creates an NPE in BorderLayout.getMax() and SWT.HORIZONTAL creates an NPE in BorderLayout.getTotal().

Also, what would be the expected behaviour of pack()? I know BorderLayout has some spacing/margin dimensions specified, so is pack() a valid call with BorderLayout?

from eclipse.platform.swt.

I don't see a reason why pack() should not be supported.

from eclipse.platform.swt.

Alright, still looking into this, no patch yet.

from eclipse.platform.swt.