Thank you very much!
ZhangMaosen
-邮件原件-
发件人: Pavel Stehule [mailto:pavel.steh...@gmail.com]
发送时间: 2009年8月24日 10:54
收件人: 张中
抄送: pgsql-hackers@postgresql.org
主题: Re: [HACKERS] help:how to write a planner_hook and other hooks in pg
Hello
http://archives.postgresql.org/pgsql-patches/2007-05
On Sun, Aug 23, 2009 at 11:16 AM, Kushal Vaghani wrote:
> I have downloaded and built the 8.2.4 postgreSQL from source. This was done
> by running the build.bat file under src\tools\msvc. I do get a few errors
> with some contrib projects, but I do not care about them.
>
> When i do install(install
Hello
http://archives.postgresql.org/pgsql-patches/2007-05/msg00421.php
regards
Pavel Stehule
2009/8/24 张中 :
> Hi all:
>
> I am doing some experiments programming on PG. I tried to modify planner and
> I find the ‘planner_hook’ in source code, but I don’t know how to use it.
>
> Anybody knows? I
Hi all:
I am doing some experiments programming on PG. I tried to modify planner and
I find the ‘planner_hook’ in source code, but I don’t know how to use it.
Anybody knows? If any code sample, very precious.
Thanks
张茂森 ZhangMaosen
Hello,
I have downloaded and built the 8.2.4 postgreSQL from source. This was done
by running the build.bat file under src\tools\msvc. I do get a few errors
with some contrib projects, but I do not care about them.
When i do install(install.pl), there is a destination directory created. Now
my qu
On 24 Aug 2009, at 00:15, Greg Stark wrote:
On Sun, Aug 23, 2009 at 11:16 PM, Grzegorz
Jaskiewicz wrote:
ok folks, here's the last one for Today:
http://zlew.org/postgresql_static_check/scan-build-2009-08-23-29/
This does look better. The first one I looked at looks like a
legitimate bug. T
On Sun, Aug 23, 2009 at 11:16 PM, Grzegorz
Jaskiewicz wrote:
> ok folks, here's the last one for Today:
>
> http://zlew.org/postgresql_static_check/scan-build-2009-08-23-29/
This does look better. The first one I looked at looks like a
legitimate bug. The nice thing is that this seems to be pickin
Roger Leigh escribió:
> +#if (defined(HAVE_LANGINFO_H) && defined(CODESET))
> + if (!strcmp(nl_langinfo(CODESET), "UTF-8"))
> + text_format = &utf8format;
> +#endif
I think you should also try to match to "UTF8", and do it in
case-insensitive manner (pg_strcasecmp), just like chkl
Roger Leigh wrote:
> On Sun, Aug 23, 2009 at 11:47:02AM -0400, Alvaro Herrera wrote:
> > Wouldn't it be better to do a single fprintf call here instead of
> > fputc + fputs?
>
> It's certainly possible to change it; the above might be slightly more
> efficient than a call to fprintf since you ski
ok folks, here's the last one for Today:
http://zlew.org/postgresql_static_check/scan-build-2009-08-23-29/
tar ball with report can be downloaded from here: http://zlew.org/postgresql_static_check/postgresql_static_check_23rdAugust2009.tar.xz
(compressed with lzma's xz tool).
here's the pat
On 23 Aug 2009, at 20:31, Tom Lane wrote:
Grzegorz Jaskiewicz writes:
for the record, here's patch that marks elog, etc as dead ends:
That does not look like the right thing at all, since now the checker
will believe that elog(NOTICE) and such don't return. I think you
need to use Martijn'
Grzegorz Jaskiewicz writes:
> for the record, here's patch that marks elog, etc as dead ends:
That does not look like the right thing at all, since now the checker
will believe that elog(NOTICE) and such don't return. I think you
need to use Martijn's suggestion instead.
"Todd A. Cook" writes:
> Tom Lane wrote:
>>> If you roll back a truncate, do you get the expected state?
> I did a number of variations on the test below, with and without "on drop
> commit",
> and similar tests where the "create table" is done before the "begin". After
> the
> checkpoint, the
this one should contain substantialy less false positives, because
error functions were marked as the 'never exit' points:
http://zlew.org/postgresql_static_check/scan-build-2009-08-23-9/
for the record, here's patch that marks elog, etc as dead ends:
Index: src/include/utils/elog.h
==
Roger Leigh wrote:
> Convert print_aligned_text, and its helper function, to use
> table formatting in place of hardcoded ASCII characters.
> @@ -841,7 +856,10 @@ print_aligned_text(const printTableContent *cont, const
> printTextFormat *format,
>
> /* left border */
>
memory leak:
http://zlew.org/postgresql_static_check/scan-build-2009-08-23-5/report-46wcmJ.html#EndPath
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers
Greg Stark writes:
> On Sun, Aug 23, 2009 at 6:11 PM, Grzegorz Jaskiewicz
> wrote:
>> http://zlew.org/postgresql_static_check/scan-build-2009-08-23-5/report-MAVb5D.html#EndPath
>> for a very positive one - at least from strict language point of view.
>>
>> consider: float f = 1; f++; pri
about the false positives around 'null reference'. I'll try sticking
exit(1)'s at the end of each macro - and see if that makes most of
them go away.
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref
On Sun, Aug 23, 2009 at 6:11 PM, Grzegorz Jaskiewicz
wrote:
> http://zlew.org/postgresql_static_check/scan-build-2009-08-23-5/report-MAVb5D.html#EndPath
> for a very positive one - at least from strict language point of view.
>
> consider: float f = 1; f++; printf("%f\n", f);
I believe th
http://zlew.org/postgresql_static_check/scan-build-2009-08-23-5/report-MAVb5D.html#EndPath
for a very positive one - at least from strict language point of view.
consider: float f = 1; f++; printf("%f\n", f);
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To mak
On 23 Aug 2009, at 17:41, Greg Stark wrote:
Do you know how to teach clang about functions which never return?
http://clang-analyzer.llvm.org/annotations.html#attr_noreturn
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://ww
On Sun, Aug 23, 2009 at 05:41:24PM +0100, Greg Stark wrote:
> On Sun, Aug 23, 2009 at 4:57 PM, Grzegorz Jaskiewicz
> wrote:
> > I am sure there's plenty of false positives, but I am also quite sure
> > there's many real errors on that list.
>
> Do you know how to teach clang about functions which
On 23 Aug 2009, at 17:41, Greg Stark wrote:
On Sun, Aug 23, 2009 at 4:57 PM, Grzegorz Jaskiewicz> wrote:
I am sure there's plenty of false positives, but I am also quite sure
there's many real errors on that list.
Do you know how to teach clang about functions which never return?
That seems t
On Sun, Aug 23, 2009 at 11:47:02AM -0400, Alvaro Herrera wrote:
> Roger Leigh wrote:
> > Convert print_aligned_text, and its helper function, to use
> > table formatting in place of hardcoded ASCII characters.
>
> > @@ -841,7 +856,10 @@ print_aligned_text(const printTableContent *cont,
> > const
On Sun, Aug 23, 2009 at 4:57 PM, Grzegorz Jaskiewicz
wrote:
> I am sure there's plenty of false positives, but I am also quite sure
> there's many real errors on that list.
Do you know how to teach clang about functions which never return?
That seems to be causing most of the false positives beca
So,
after successful, and helpful Saturday with llvm's clang static
checker, I decided to run it against postgresql's source code.
Result can be seen at: http://zlew.org/postgresql_static_check/scan-build-2009-08-23-5/
.
One directory below is the tar file, with the report.
I am sure there'
On Sun, Aug 23, 2009 at 1:57 AM, Tom Lane wrote:
> Robert Haas writes:
>> I posted a note about a week ago which drew far less commentary than I
>> expected, regarding the timetable for the release of 8.5.
>
>> http://archives.postgresql.org/pgsql-hackers/2009-08/msg01256.php
>
>> Perhaps this is
On Sun, Aug 23, 2009 at 01:49:02AM +0100, Roger Leigh wrote:
> On Sat, Aug 22, 2009 at 07:42:10PM -0400, Robert Haas wrote:
> > On Sat, Aug 22, 2009 at 2:13 PM, Roger Leigh wrote:
> > > Further minor cleanups to tweak column alignment in a corner case,
> > > and to correct indentation to match the
2009/8/23 Magnus Hagander :
> For another data point, Microsoft documentation says:
> "This POSIX function is deprecated beginning in Visual C++ 2005. Use
> the ISO C++ conformant _hypot instead."
>
> _hypot() has been there since Windows 95, so it shouldn't be a problem
> to use it - it just nee
On Sun, Aug 23, 2009 at 07:42, Tom Lane wrote:
> Greg Stark writes:
>> If there's a performance advantage then we could add a configure test
>> and define the macro to call hypot(). You said it existed before C99
>> though, how widespread was it? If it's in all the platforms we support
>> it might
Tom Lane wrote:
> Greg Stark writes:
>
>> If there's a performance advantage then we could add a configure test
>> and define the macro to call hypot(). You said it existed before C99
>> though, how widespread was it? If it's in all the platforms we support
>> it might be reasonable to just go
31 matches
Mail list logo