Modernize and clean up petscsys.h

- Deprecate PETSC_NULL.
- Switch from preprocessor defines to anonymous enums for constant expressions.
- Surround macro arguments in ().
- Remove redundant casts.

Modernize and clean up petscsys.h

- Deprecate PETSC_NULL.
- Switch from preprocessor defines to anonymous enums for constant expressions.
- Surround macro arguments in ().
- Remove redundant casts.
- Clean up overly complex preprocessor defines in various PetscMem*() functions.

show more ...

Merge branch 'barry/2022-12-27/rm-vecgetarray-fortran-examples' into 'main'

Replace use of VecGetArray() and friends from Fortran examples and docs

See merge request petsc/petsc!5950

Deprecate VecGetArray() and similar routines in favor of VecGetArrayF90() and similar routines, and update Docs

Commit-type: housekeeping
/spend 1h

Add MatMPIAIJGetSeqAIJF90(), MatMPIAIJRestoreSeqAIJF90()

Merge branch 'jolivet/feature-removed-errorchecking' into 'main'

Remove remaining traces of --with-errorchecking

See merge request petsc/petsc!5962

Fix -Wunused-parameter for CUDA files

Remove remaining traces of --with-errorchecking

Fix -Wunused-parameter and -Wimplicit-fallthrough

Merge remote-tracking branch 'origin/release'

Fix some typos

remove hardwired array in mpiaij

Merge remote-tracking branch 'origin/release'

Fix compiler warning caused by having two variables with the same name

mpiov.c(57): warning #823: reference is to variable "i" (declared at line 32) -- under old for-init scoping rules it would have

Fix compiler warning caused by having two variables with the same name

mpiov.c(57): warning #823: reference is to variable "i" (declared at line 32) -- under old for-init scoping rules it would have been variable "i" (declared at line 47)
PetscCall(ISRestoreIndices(is[i], &idx));
^

show more ...

Make MatIncreaseOverlap_AIJ respect blocking

fix matconvert aij to baij to correctly check for any nonzero in a block to do proper BAIJ preallocation

Remove PETSC_SKIP_SPINLOCK

I don't see the reason why we needed it in the first place
Disabling the SPINLOCK makes some parts of the PETSc code base not thread-safe

Merge remote-tracking branch 'origin/release'

Mat: fix a bug when calling MatCreateSubMatrix() on a mpiaijkokkos matrix

The created submatrix would have A of seqaijkokkos, but B of seqaij

Reported-by: "Fackler, Philip" <facklerpw@ornl.gov>

Register matcovert_seqaijhipsparse in HIP implementation

Merge remote-tracking branch 'origin/release'

Remove unneeded " on xxx" from most Collective comments in manual pages

The phrase was unneeded 99% of the time

I did not remove in the dm directory since another MR is still active that affects th

Remove unneeded " on xxx" from most Collective comments in manual pages

The phrase was unneeded 99% of the time

I did not remove in the dm directory since another MR is still active that affects that directory

Commit-type: documentation
/spend 20m

show more ...

Use PetscHMapISet() in place of PetscHMapISetWithMode(INSERT_VALUES) as the intent is clearer

- Replace PetscTable with PetscHMapI.
- Rename:
- PetscTableCreate() -> PetscHMapICreateWithSize()
- PetscTableFind() -> PetscHMapIGetWithDefault()
- PetscTableAdd() -> PetscHMapISetWithMode()

Merge remote-tracking branch 'origin/release'

Fix missing "Key" and incorrect "Command" for manual page listing of Options Database Key(s):

Commit-type: documentation
/spend 10m