From: | Alfranio Correia Junior <alfranio(at)lsd(dot)di(dot)uminho(dot)pt> |
---|---|
To: | Alvaro Herrera <alvherre(at)surnet(dot)cl> |
Cc: | pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: HOOKS for Synchronous Replication |
Date: | 2005-06-21 09:35:32 |
Message-ID: | 42B7DF64.6050705@lsd.di.uminho.pt |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
>First of all your patch does not conform with the project style. Please
>have a look at how other files are indented, in particular regarding
>brace position and ereport() arguments (any function arguments really,
>but in ereport your problems are more visible). Also, always use
>ereport() for user messages, elog() for conditions that involve
>can't-happen situations (server bugs, like not finding a tuple in a
>catalog that should be there, etc).
>
>
I think it is ok now.
However, I corrected the indentation manually.
I could not run some of the tools, namely the "entab".
/usr/lib/gcc-lib/i386-redhat-linux/3.3.3/include/varargs.h:4:2: #error
"GCC no longer implements <varargs.h>."
/usr/lib/gcc-lib/i386-redhat-linux/3.3.3/include/varargs.h:5:2: #error
"Revise your code to use <stdarg.h>."
halt.c:23: error: syntax error before "va_dcl"
halt.c:24: error: syntax error before '{' token
----
Linux alfranio.lsd.di.uminho.pt 2.6.8-1.521 #1 Mon Aug 16 09:01:18 EDT
2004 i686 i686 i386 GNU/Linux
(GCC) 3.3.3 20040412 (Red Hat Linux 3.3.3-7)
Do you have any idea ?
Best regards,
Alfranio Junior.
From | Date | Subject | |
---|---|---|---|
Next Message | Andreas Pflug | 2005-06-21 09:40:58 | Re: Server instrumentation |
Previous Message | Oleg Bartunov | 2005-06-21 09:02:07 | Re: Schedule for 8.1 feature freeze |