[petsc-dev] please review this patch

[Satish Balay]

They were added in the following commit

http://petsc.cs.iit.edu/petsc/petsc-dev/rev/97e68eb8972d

Satish

On Wed, 16 May 2012, Barry Smith wrote:

> 
>   Looks good for me. I cannot explain why those lines are there.
> 
>    Barry
> 
> On May 16, 2012, at 10:51 AM, Lisandro Dalcin wrote:
> 
> > Could you please help me to review this patch? These lines disable the
> > usage of MatSolve_SeqAIJ_NaturalOrdering_inplace that is set a few
> > lines above. Am I missing something?
> > 
> > 
> > diff -r 7ea93c40c86e src/mat/impls/aij/seq/aijfact.c
> > --- a/src/mat/impls/aij/seq/aijfact.c	Sun May 13 21:38:46 2012 -0500
> > +++ b/src/mat/impls/aij/seq/aijfact.c	Wed May 16 12:34:43 2012 -0300
> > @@ -759,8 +759,6 @@
> >       ierr = PetscInfo2(A,"number of shift_nz tries %D, shift_amount
> > %G\n",sctx.nshift,sctx.shift_amount);CHKERRQ(ierr);
> >     }
> >   }
> > -  (C)->ops->solve            = MatSolve_SeqAIJ_inplace;
> > -  (C)->ops->solvetranspose   = MatSolveTranspose_SeqAIJ_inplace;
> >   ierr = Mat_CheckInode(C,PETSC_FALSE);CHKERRQ(ierr);
> >   PetscFunctionReturn(0);
> > }
> > 
> > 
> > -- 
> > Lisandro Dalcin
> > ---------------
> > CIMEC (INTEC/CONICET-UNL)
> > Predio CONICET-Santa Fe
> > Colectora RN 168 Km 472, Paraje El Pozo
> > 3000 Santa Fe, Argentina
> > Tel: +54-342-4511594 (ext 1011)
> > Tel/Fax: +54-342-4511169
> 
>