From: wpp@marie.physik.tu-berlin.de (Kai Petzke)
Message-Id: <9407100853.AA27673@marie.physik.tu-berlin.de>
Subject: Re: postgres v4r2 for linux
To: corey@bbs.xnet.com (Corey Sweeney)
Date: Sat, 9 Jul 1994 23:08:00 +0200 (MET DST)
Cc: linux-postgres@native-ed.bc.ca
In-Reply-To: <9407030438.AA04e2p@bbs.xnet.com> from "Corey Sweeney" at Jul 2, 94 10:38:52 pm
Content-Type: text
Content-Length: 1729

Corey Sweeney (corey@bbs.xnet.com) said:

> Hey! it finnally happened.  Now we can get some real work done :)
> 
> Ok here's my proposal for the plan.  (it's the same plan as before, with a
> couple of additions of my own)
> 
> 
> ok, first we get it working with -Wall.  (without warnings)

I think so, too.  I hope, that we can parallelize this task.
Everybody who wants to help catches a directory, and tries to
get all the files in it compiled without warnings.

I want to do the beginning: I choose ~postgres/src/backend/planner.

There are two problems, though:

a) Casts to a superclass.  C does no implicit casting from a
   subclass to a superclass, C++ does.  To turn of the C compiler
   warnings would require to add casts, which are bad for C++,
   though, because they inhibit type checking.

   I see the following alternatives:
   - ignore the C warnings about illegal pointers passed to functions.
   - add the casts, as a macro call.  Eg:
     #ifdef  __cplusplus
     #define super(x) (x)		/* C - Version */
     #define super(x) 			/* C++ - Version */
     #endif

     foo(super(Node) arg1, arg2);

     That would call the function foo() with arg1 converted to type
     Node (the base type of the Postgres), and arg2 unmodified.

b) Communication.  Does anybody know of a method to keep several
   world wide distributed versions of an RCS log identical?

   I guess, not, so we will have to go for something less
   sophisticated.  I can volunteer to collect patches.  E-mail is
   quite reliable here, despite the ftp problems.  The bad thing
   is for modifications to header files, because they affect
   everybody's work.  Maybe patches to headers should be posted
   to the list immediately.


Kai