<div>All public API changes have to be documented at src/docs/website/documentation/changes/dev.html.</div><div><br></div><a href="http://petsc.cs.iit.edu/petsc/petsc-dev/rev/a942f19a37c9">http://petsc.cs.iit.edu/petsc/petsc-dev/rev/a942f19a37c9</a><div>
<br></div><div>Some questions and comments on this patch:</div><div><br></div><div>1. Why don't you use the API (e.g. PetscLayoutSetBlockSize()) when setting the block size?</div><div><br></div><div>2. What was wrong with the default MatGetVecs()?</div>
<div><br></div><div>3. Options prefixes should inherit the prefix from the parent and should end with "_" so that they aren't squashed together with the suboptions.</div><div><br></div><div>4. MatCreateIS() leaks the new Vec "global".</div>
<div><br></div><div>5. Why is the matrix being forced to be square?</div><div><br></div><div>6. Why is the size of the rmap/cmap being changed to be possibly different from m,n,M,N passed in? By not using the PetscLayout API, it looks like we can end up with a corrupt PetscLayout this way.</div>