search

MPAS-Dev / MPAS

Repository for private MPAS development prior to the MPAS v6.0 release.
Other
4 stars 0 forks source link

Change '%i' to '$i' in call to mpas_log_write from mpas_block_decomp_cells_for_proc #1545

Closed mgduda closed 6 years ago

mgduda commented 6 years ago

This merge corrects a message format string to use '$i' rather than '%i' in the mpas_block_decomp_cells_for_proc routine.

If a mesh decomposition file with the right number of partitions is not found, the message string should use '$i' rather than '%i' to print an integer (which in this case is the number of requested blocks). C format strings use '%i', while the MPAS log module uses '$i'.

mgduda commented 6 years ago

I think I've also seen this a few times in the past, but I also never bothered to create a PR to fix it. I suppose this is one of the benefits of releases: they give motivation to fix those small issues that are too easy to otherwise ignore.

I'll merge this in just a few minutes.