Profiling: Improve !defined(PETSC_USE_LOG) #defines

- If we cast macro arguments to void, we can avoid unused errors that have
forced lots of conditionally included code like this:

```c
#if def

Profiling: Improve !defined(PETSC_USE_LOG) #defines

- If we cast macro arguments to void, we can avoid unused errors that have
forced lots of conditionally included code like this:

```c
#if defined(PETSC_USE_LOG)
PetscLogEvent event;
#endif
```

- Having global counters like petsc_TotalFlops externed event
if !PetscDefined(USE_LOG) helps to remove a lot of needless
`#ifdef`s around code that can instead be placed in
`if (PetscDefined(USE_LOG)) {}` blocks.

show more ...

No MatDummy in MatCreateSubMatrices_SeqSBAIJ()

Rename PetscValidPointer -> PetscAssertPointer

Deploy new PetscValidPointer, remove PetscValidXXXPointer

MatEliminateZeros(): add [S]BAIJ implementations

MatAssemblyEnd(): same logic for [S]BAIJ than for AIJ

MatSeqSBAIJ: remove unused code

It has been unused for 10+ years

Manual linter fixes: mat

Fix -Wextra-semi-stmt

Lint apply: mat

Merge branch 'barry/2023-06-15/add-fortran-file-viewer' into 'main'

Allow one to attach a PETSCVIEWERASCII to a Fortran file unit

See merge request petsc/petsc!6607

HAVE_FORTRAN should be USE_FORTRAN_BINDINGS since it is about generating the Fortran bindings, not about if the Fortran compiler exists

Merge remote-tracking branch 'origin/release'

Merge branch 'jolivet/fix-minor' into 'release'

PCFIELDSPLIT: fix -ksp_view_pre (+ minor typo)

See merge request petsc/petsc!6613

Fix make checkbadSource

triggered with a merge of !6507 (main) and !6612 (release) changes

Merge remote-tracking branch 'origin/release'

Fix errors introduced in !6090 reported in https://gitlab.com/petsc/petsc/-/issues/1389

If matrices were destroyed (possibly through a type change) before their hash state was destroyed they would

Fix errors introduced in !6090 reported in https://gitlab.com/petsc/petsc/-/issues/1389

If matrices were destroyed (possibly through a type change) before their hash state was destroyed they would result in incorrect results or memory leaks

Commit-type: bug-fix
Reported-by: Pierre Jolivet

show more ...

PCFIELDSPLIT: fix -ksp_view_pre (+ minor typo)

minor: Fix typo nozero -> nonzero

Fix gcc-13 warnings found by Pierre

Based-on-patch-by: Pierre Jolivet <pierre@joliv.et>

Merge remote-tracking branch 'origin/release'

Don't use PetscMemcpy() or PetscArraycpy() to copy the ops struct