Hey Ivan,

Your patch leaves two blocks of unreachable code.

Also, layout is extremely tricky and easily testable, so EVERY change to 
default/table/flow layout MUST have a test case.  See TableLayoutTest.cs.

Other than that, it looks ok.

Jonathan


Ivan N. Zlatev wrote:
> Hey,
> 
> Attached patch fixes bug 355408.  We have to take in account controls
> and their size regardless of their dock/anchor state when calculating
> row/column sizes.
> 
> If there are no objections I will commit.
> 
> 
> ------------------------------------------------------------------------
> 
> _______________________________________________
> Mono-winforms-list maillist  -  [email protected]
> http://lists.ximian.com/mailman/listinfo/mono-winforms-list

_______________________________________________
Mono-winforms-list maillist  -  [email protected]
http://lists.ximian.com/mailman/listinfo/mono-winforms-list

Reply via email to