[petsc-dev] change to VecView_MPI_ASCII() major affect on nightly builds

[Barry Smith]

   Jed,

     Your call to PetscObjectGetName() is causing major changes to example output 

77ae766c src/vec/vec/impls/mpi/pdvec.c (Jed Brown            2013-12-21 18:23:45 -0700   54)   ierr = PetscObjectGetName((PetscObject)xin,&name);CHKERRQ(ierr);
77ae766c src/vec/vec/impls/mpi/pdvec.c (Jed Brown            2013-12-21 18:23:45 -0700   55)   ierr = PetscViewerGetFormat(viewer,&format);CHKERRQ(ierr);

because now all Vecs have names when printed so nightly tests are filled with 


1c1
< Vec Object: 2 MPI processes
---
> Vec Object:Vec_0x84000005_0 2 MPI processes
/home/balay/petsc.clone-3/src/vec/vec/examples/tests
Possible problem with with ex3_1, diffs above
=========================================
1c1
< Vec Object: 2 MPI processes
---
> Vec Object:Vec_0x84000004_0 2 MPI processes
/home/balay/petsc.clone-3/src/vec/vec/examples/tests
Possible problem with with ex9_1, diffs above
=========================================


How do we want to deal with this? Normally in the past Vec’s never had names. Could you put the PetscObjectGetName() back in a place where it only gets called when truly needed like when viewer with Matlab instead of all the time? Note that it still only gets called when needed in VecView_Seq_ASCII.

   Thanks

    Barry

Request-assigned: Jed VecView_MPI_ASII() always prints Vector name now resulting in messy nightlys