<div dir="ltr">Thanks, Satish,<div><br></div><div>I updated the changes.  Please check again.</div><div><br></div><div><br></div><div>Fande,</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Jul 16, 2018 at 12:22 PM, Satish Balay <span dir="ltr"><<a href="mailto:balay@mcs.anl.gov" target="_blank">balay@mcs.anl.gov</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Mon, 16 Jul 2018, Fande Kong wrote:<br>
<br>
> In order to have make MatRestoreArray_HYPRE and MatRestoreArray_HYPRE looks<br>
> same.<br>
<br>
</span>Well its more like PetscMalloc() and PetscFree() to me - so they don't have matching requirements..]<br>
<span class=""><br>
<br>
> If did not do anything to size and array, and then we may have some<br>
> warnings: unused parameters.<br>
> <br>
> Should we do something like this?<br>
> <br>
> +static PetscErrorCode MatRestoreArray_HYPRE(Mat A, PetscInt /*size*/, void<br>
> **/*array*/)<br>
> +{<br>
> +  Mat_HYPRE          *hA = (Mat_HYPRE*)A->data;<br>
> +<br>
> +  PetscFunctionBegin;<br>
> +  hA->available = PETSC_TRUE;<br>
> +  PetscFunctionReturn(0);<br>
> +}<br>
<br>
<br>
</span>I think its should be:<br>
<br>
static PetscErrorCode MatRestoreArray_HYPRE(Mat A, void **array)<br>
<span class="HOEnZb"><font color="#888888"><br>
Satish<br>
</font></span></blockquote></div><br></div>