[PATCHES] cancel <3FB0B7CB.2070602@bigfoot.com>

2003-11-11 Thread mendola
This message was cancelled from within Mozilla. ---(end of broadcast)--- TIP 3: if posting/reading through Usenet, please send an appropriate subscribe-nomail command to [EMAIL PROTECTED] so that your message can get through to the maili

[PATCHES] cancel <3FB0C0CE.6050007@bigfoot.com>

2003-11-11 Thread mendola
This message was cancelled from within Mozilla. ---(end of broadcast)--- TIP 2: you can get off all lists at once with the unregister command (send "unregister YourEmailAddressHere" to [EMAIL PROTECTED])

[PATCHES] mcxt.c

2003-09-07 Thread Mendola Gaetano
A test for null string is missing here: *** pgsql/src/backend/utils/mmgr/mcxt.c 2003-09-02 13:30:05.0 +0200 --- pgsql.old/src/backend/utils/mmgr/mcxt.c 2003-08-04 04:40:08.0 +0200 *** char * *** 620,632 MemoryContextStrdup(MemoryContext context, const char *strin

Re: [PATCHES] mcxt.c

2003-09-08 Thread Gaetano Mendola
"Tom Lane" <[EMAIL PROTECTED]> wrote: > "Mendola Gaetano" <[EMAIL PROTECTED]> writes: > > A test for null string is missing here: > > > MemoryContextStrdup(MemoryContext context, const char *string) > > { > > char *nstr; > > -

Re: [PATCHES] mcxt.c

2003-09-08 Thread Gaetano Mendola
"Neil Conway" <[EMAIL PROTECTED]> wrote: > On Mon, 2003-09-08 at 11:09, Gaetano Mendola wrote: > > "Tom Lane" <[EMAIL PROTECTED]> wrote: > > > I see no value at all in an assert. The code will dump core just fine > > > with or wi

Re: [PATCHES] mcxt.c

2003-09-08 Thread Gaetano Mendola
"Tom Lane" <[EMAIL PROTECTED]> wrote: > "Gaetano Mendola" <[EMAIL PROTECTED]> writes: > > "Tom Lane" <[EMAIL PROTECTED]> wrote: > >> This seems inappropriate to me. Are you going to suggest that every > >> routine that

[PATCHES] strdup => xstrdup patche

2003-09-16 Thread Mendola Gaetano
Hi, I think that I cover know all strdup not checked on the tree backend. I'm going to investigate now for malloc unchecked. Regards Gaetano Mendola psql.patch Description: Binary data ---(end of broadcast)--- TIP 6: Have you searched our

Re: [PATCHES] strdup => xstrdup patche

2003-09-16 Thread Mendola Gaetano
""Mendola Gaetano"" <[EMAIL PROTECTED]> wrote: > Hi, > I think that I cover know all strdup not checked > on the tree backend. Of course I forgot to remove the null pointer check inside the xstrdup, please use this patch instead. Regards Gaetano Mendola

[PATCHES] Waiting the patch applied

2003-09-26 Thread Gaetano Mendola
Hi, one question just to know the right action to take: After having submitted a patch in order to work on a new patch shall I open another fresh CVS snapshot or shall I work on top of my old patch not yet applied ( or refused )? Regards Gaetano Mendola ---(end of broadcast

Re: [PATCHES] Waiting the patch applied

2003-09-27 Thread Gaetano Mendola
Peter Eisentraut wrote: Gaetano Mendola writes: one question just to know the right action to take: After having submitted a patch in order to work on a new patch shall I open another fresh CVS snapshot or shall I work on top of my old patch not yet applied ( or refused )? Since applying the

Re: [PATCHES] psql italian translation

2003-10-06 Thread Gaetano Mendola
a una tabella Sinstassi: DROP TABLE nome [, ...] [ CASCADE | RESTRICT ] Please see my post ( I choose the wron address) on hackers, there are some typos on the traslation. Regards Gaetano Mendola ---(end of broadcast)--- TIP 8: explain analyze is your friend

Re: [PATCHES] minor HISTORY improvement

2003-10-16 Thread Gaetano Mendola
ble in this way: my_variable ALIAS FOR$1 (note the space missing between FOR and $1). Regards Gaetano Mendola ---(end of broadcast)--- TIP 1: subscribe and unsubscribe commands go to [EMAIL PROTECTED]

Re: [PATCHES] minor HISTORY improvement

2003-10-17 Thread Gaetano Mendola
Bruce Momjian wrote: Gaetano Mendola wrote: Bruce Momjian wrote: Thanks. I have applied this to the SGML docs. The HISTORY file will be updated automatically now. I didn't see in the notes about the "Migration to version 7.4" the fact that now inside a plpgsql function

[PATCHES] pg_autovacuum README patch

2003-10-20 Thread Gaetano Mendola
--- README.good 2003-10-21 02:40:19.0 +0200 +++ README 2003-10-21 02:37:43.0 +0200 @@ -1,3 +1,3 @@ -PG_autovacuum README +pg_autovacuum README @@ -148,8 +148,8 @@ vacuum analyze is performed. -AnalyzeThreshold is equal to: +deleteThreshold is equa

Re: [PATCHES] pg_autovacuum README patch

2003-10-26 Thread Gaetano Mendola
Gaetano Mendola wrote: --- README.good 2003-10-21 02:40:19.0 +0200 +++ README 2003-10-21 02:37:43.0 +0200 @@ -1,3 +1,3 @@ -PG_autovacuum README +pg_autovacuum README @@ -148,8 +148,8

Re: [PATCHES] equal() perf tweak

2003-11-05 Thread Gaetano Mendola
plementation. And another my wish is too see Postgres compilable with a c++ compiler so a day we can use ( in a absolutely portable way): vector, list, map already implemented, considering also the fact that all stl container are "memory allocator" customizable, for example you can use

Re: [PATCHES] equal() perf tweak

2003-11-05 Thread Gaetano Mendola
Neil Conway wrote: Gaetano Mendola <[EMAIL PROTECTED]> writes: Why instead of reinvent the whell not use, or at least do a "C" port of stl::list ? Because (a) implementing a linked list is pretty trivial (b) the only difficult part is getting the semantics / API right. I d

Re: [PATCHES] equal() perf tweak

2003-11-06 Thread Gaetano Mendola
Tom Lane wrote: Gaetano Mendola <[EMAIL PROTECTED]> writes: [ use list with master node and circular linking ] now is very late here ( 04:17 AM ) so I wrote the wrong code ( not checked ) but show the idea of avoid "if". This saves an "if" during addition of a

Re: [PATCHES] equal() perf tweak

2003-11-06 Thread Gaetano Mendola
Tom Lane wrote: Gaetano Mendola <[EMAIL PROTECTED]> writes: [ use list with master node and circular linking ] now is very late here ( 04:17 AM ) so I wrote the wrong code ( not checked ) but show the idea of avoid "if". This saves an "if" during addition of a

Re: [PATCHES] SIGPIPE handling, take two.

2003-11-11 Thread Gaetano Mendola
conn->do_sigaction = false; if ( err == 0 ) sigaction(SIGPIPE, &oact, NULL); } #endif /* !HAVE_POSIX_SIGNALS */ Regards Gaetano Mendola ---(end of broadcast)

Re: [PATCHES] SIGPIPE handling, take two.

2003-11-11 Thread Gaetano Mendola
Gaetano Mendola wrote: Bruce Momjian wrote: I think this is the patch I like. It does the auto-detect handling as I hoped. I will just do the doc updates to mention it. My only issue is that this is per-connection, while I think you have to create a global variable that defaults to false, and

Re: [PATCHES] equal() perf tweak

2003-11-12 Thread Gaetano Mendola
Bruce Momjian wrote: Is there a TODO here? I'm just evaluating the performances of two version the one proposed by Neil and the one in a "stl::list" fashion. Tom suggested also to see if is the case to implement the function size() constant time or not. Regards

[PATCHES] New List

2003-11-13 Thread Gaetano Mendola
ser 0m3.980s sys0m1.090s So at the end I think that the best solution is the Neil classical proposal and mantain the length function constant time. Regards Gaetano Mendola ---(end of broadcast)--- TIP 8: explain analyze is your friend

Re: [PATCHES] New List

2003-11-15 Thread Gaetano Mendola
Neil Conway wrote: Gaetano Mendola <[EMAIL PROTECTED]> writes: 10E6 INSERT => real 0m5.161s user 0m4.010s sys0m1.150s What operation is this benchmarking? Only linked-list appends, or something else? Only the head append.

Re: [PATCHES] pg_generate_sequence and info_schema patch (Was: SELECT

2004-02-04 Thread Gaetano Mendola
the result set is empty IMHO this is the more natural behavior. my 2 cents. Regards Gaetano Mendola -BEGIN PGP SIGNATURE- Version: GnuPG v1.2.3 (MingW32) Comment: Using GnuPG with Thunderbird - http://enigmail.mozdev.org iD8DBQFAIYxo7UpzwH2SGd4RAnr9AJ4//4AO9VaIQmweneB0BJe8Dv

Re: [PATCHES] updated list rewrite

2004-05-24 Thread Gaetano Mendola
Neil Conway wrote: I've attached a new version of the list rewrite patch (gzip'ed). Changes since the last patch: I never dig on postgresql code in order to see how and where lists are used, I'm wondering if also a "rope" could be usefull somewhere

Re: [PATCHES] pg_ctl.c

2004-05-25 Thread Gaetano Mendola
isplay. Regards Gaetano Mendola *** pg_ctl.c2004-05-26 02:48:38.0 +0200 --- pg_ctl.c.orig 2004-05-26 02:43:32.0 +0200 *** *** 26,33 /* postmaster version ident string */ #define PM_VERSIONSTR "postmaster (PostgreSQL) " PG_VERSION "\n"

Re: [PATCHES] pg_ctl.c

2004-05-26 Thread Gaetano Mendola
Andrew Dunstan wrote: Gaetano Mendola wrote: Bruce Momjian wrote: however below the result of my quich review: 1) exit(1) => exit(EXIT_FAILURE) If we used a number of different error codes I might agree. But it seems pointless here, and the style is widely used in our code base (I just coun

Re: [PATCHES] pg_ctl.c

2004-05-26 Thread Gaetano Mendola
Andrew Dunstan wrote: Gaetano Mendola wrote: Andrew Dunstan wrote: Gaetano Mendola wrote: Bruce Momjian wrote: however below the result of my quich review: 1) exit(1) => exit(EXIT_FAILURE) If we used a number of different error codes I might agree. But it seems pointless here, and the style

Re: [PATCHES] pg_ctl.c

2004-06-02 Thread Gaetano Mendola
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Bruce Momjian wrote: | Gaetano Mendola wrote: | |>Bruce Momjian wrote: |> |> |>>Here is the C version of pg_ctl.c written by Andrew Dunstan and updated |>>by me. |>> |>>You can use it by creating a src/bin/pg_ctl_test d

Re: [PATCHES] psql - missing tab-completion support for tablespaces

2004-08-09 Thread Gaetano Mendola
was not able to retrieve the "active" savepoints, is it really impossible to hack that part ? Regards Gaetano Mendola ---(end of broadcast)--- TIP 5: Have you checked our extensive FAQ? http://www.postgresql.org/docs/faqs/FAQ.html

Re: [PATCHES] [HACKERS] 8.0 libpq: missing get_home_path()

2004-08-19 Thread Gaetano Mendola
Bruce Momjian wrote: OK, patch attached and applied. I also updated the docs. Just a note for the docs: "%USERPROFILE%/.pgpass" shall be not "%USERPROFILE%\.pgpass" ? Regards Gaetano Mendola ---(end of broadcast)--- TIP 1: subsc

Re: [PATCHES] psql tab-completion improvements

2004-09-10 Thread Gaetano Mendola
t low impact. Is there a "low impact" concept that override the "feature freeze" ? Regards Gaetano Mendola ---(end of broadcast)--- TIP 8: explain analyze is your friend

[PATCHES] error report on slru.c

2004-10-02 Thread Gaetano Mendola
Hi all, here a patch that 1) Rise the information about the file missing on pg_clog from detail to error 2) Insert the offset information also for a missing file I did it with the 8.0beta2 sources Regards Gaetano Mendola --- slru.c.orig 2004-10-02 10:56:01.0 +0200 +++ slru.c

Re: [PATCHES] error report on slru.c

2004-10-03 Thread Gaetano Mendola
Tom Lane wrote: Gaetano Mendola <[EMAIL PROTECTED]> writes: here a patch that 1) Rise the information about the file missing on pg_clog from detail to error This violates the message style guidelines, IMHO. Tom, your opinion is not humble but is is really appreciated, however I think

Re: [PATCHES] error report on slru.c

2004-10-03 Thread Gaetano Mendola
ow the file and the offset a transaction belong to ? Regards Gaetano Mendola ---(end of broadcast)--- TIP 5: Have you checked our extensive FAQ? http://www.postgresql.org/docs/faqs/FAQ.html

Re: [PATCHES] additional GCC warning flags

2004-10-19 Thread Gaetano Mendola
_startup_cost' may be used uninitialized in this function orindxpath.c:262: warning: 'best_total_cost' may be used uninitialized in this function = createplan.c: In function `create_plan': createplan.c:1244: warning: 'opclass' may be used unin

Re: [PATCHES] CVS should die

2004-11-04 Thread Gaetano Mendola
ver effort. The effort is not so big: http://cvs2svn.tigris.org Do not rename or move around a file because your SCM limits, is insane. Regards Gaetano Mendola ---(end of broadcast)--- TIP 2: you can get off all lists at once with the unregister comman

Re: [PATCHES] psql \! WIN32 cleanup

2004-11-06 Thread Gaetano Mendola
just do "/windows/system32/cmd.exe" and leave off the C:? The previous default was "/bin/sh" and that certainly will not work on Win32. I think another path to try if COMSPEC is not defined is: %SystemRoot%/system32/cmd.exe Regards Gaetano Mendola ---(end