Lack of new line between IF statements

Started by Fabrízio de Royes Melloalmost 7 years ago2 messages
#1Fabrízio de Royes Mello
fabriziomello@gmail.com
1 attachment(s)

Hi all,

The attached patch just a very minor adjustment to
src/bin/pg_checksums/pg_checksums.c to add new line between some IF
statements.

Regards,

--
Fabrízio de Royes Mello Timbira - http://www.timbira.com.br/
PostgreSQL: Consultoria, Desenvolvimento, Suporte 24x7 e Treinamento

Attachments:

lack_of_new_line_between_if_statement.patchtext/x-patch; charset=US-ASCII; name=lack_of_new_line_between_if_statement.patchDownload
diff --git a/src/bin/pg_checksums/pg_checksums.c b/src/bin/pg_checksums/pg_checksums.c
index f9ab27c..61bfa45 100644
--- a/src/bin/pg_checksums/pg_checksums.c
+++ b/src/bin/pg_checksums/pg_checksums.c
@@ -435,12 +435,14 @@ main(int argc, char *argv[])
 		fprintf(stderr, _("%s: data checksums are not enabled in cluster\n"), progname);
 		exit(1);
 	}
+
 	if (ControlFile->data_checksum_version == 0 &&
 		mode == PG_MODE_DISABLE)
 	{
 		fprintf(stderr, _("%s: data checksums are already disabled in cluster.\n"), progname);
 		exit(1);
 	}
+
 	if (ControlFile->data_checksum_version > 0 &&
 		mode == PG_MODE_ENABLE)
 	{
#2Michael Paquier
michael@paquier.xyz
In reply to: Fabrízio de Royes Mello (#1)
Re: Lack of new line between IF statements

On Sat, Mar 23, 2019 at 08:54:26AM -0300, Fabrízio de Royes Mello wrote:

The attached patch just a very minor adjustment to
src/bin/pg_checksums/pg_checksums.c to add new line between some IF
statements.

Thanks. This makes the code more consistent with the surroundings, so
done. At the same time I have improved the error messages in the
area as they should not have a period.
--
Michael