Re: [PATCHES] $PostgreSQL$ header -- Post 7.4 Release

2003-11-10 Thread Bruce Momjian
Neil Conway wrote:
> Rod Taylor <[EMAIL PROTECTED]> writes:
> > In src/backend/utils/adt/inet_net_ntop.c and
> > src/backend/utils/adt/inet_net_pton.c it would appear that the $Id$ tag
> > is assigned to a variable (rcsid).
> 
> ... which is never referenced AFAICT, even if the necessary
> preprocessor definitions are set so that it is seen by the compiler in
> the first place. Is there a reason not to remove this code from both
> files?

OK, removed with attached patch.  (CVS will update the file name.)

It is a BSD source code style that isn't used by our code.

-- 
  Bruce Momjian|  http://candle.pha.pa.us
  [EMAIL PROTECTED]   |  (610) 359-1001
  +  If your life is a hard drive, |  13 Roberts Road
  +  Christ can be your backup.|  Newtown Square, Pennsylvania 19073
Index: src/backend/utils/adt/inet_net_ntop.c
===
RCS file: /cvsroot/pgsql-server/src/backend/utils/adt/inet_net_ntop.c,v
retrieving revision 1.16
diff -c -c -r1.16 inet_net_ntop.c
*** src/backend/utils/adt/inet_net_ntop.c   4 Aug 2003 00:43:25 -   1.16
--- src/backend/utils/adt/inet_net_ntop.c   10 Nov 2003 19:37:34 -
***
*** 13,23 
   * PROFITS, WHETHER IN AN ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS
   * ACTION, ARISING OUT OF OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS
   * SOFTWARE.
   */
- 
- #if defined(LIBC_SCCS) && !defined(lint)
- static const char rcsid[] = "$Id: inet_net_ntop.c,v 1.16 2003/08/04 00:43:25 momjian 
Exp $";
- #endif
  
  #include "postgres.h"
  
--- 13,21 
   * PROFITS, WHETHER IN AN ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS
   * ACTION, ARISING OUT OF OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS
   * SOFTWARE.
+  *
+  *  $Header: /cvsroot/pgsql-server/src/backend/utils/adt/int.c,v 1.57 2003/08/04 
02:40:05 momjian Exp $
   */
  
  #include "postgres.h"
  
Index: src/backend/utils/adt/inet_net_pton.c
===
RCS file: /cvsroot/pgsql-server/src/backend/utils/adt/inet_net_pton.c,v
retrieving revision 1.16
diff -c -c -r1.16 inet_net_pton.c
*** src/backend/utils/adt/inet_net_pton.c   4 Aug 2003 00:43:25 -   1.16
--- src/backend/utils/adt/inet_net_pton.c   10 Nov 2003 19:37:34 -
***
*** 13,23 
   * PROFITS, WHETHER IN AN ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS
   * ACTION, ARISING OUT OF OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS
   * SOFTWARE.
   */
- 
- #if defined(LIBC_SCCS) && !defined(lint)
- static const char rcsid[] = "$Id: inet_net_pton.c,v 1.16 2003/08/04 00:43:25 momjian 
Exp $";
- #endif
  
  #include "postgres.h"
  
--- 13,21 
   * PROFITS, WHETHER IN AN ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS
   * ACTION, ARISING OUT OF OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS
   * SOFTWARE.
+  *
+  *  $Header: /cvsroot/pgsql-server/src/backend/utils/adt/int.c,v 1.57 2003/08/04 
02:40:05 momjian Exp $
   */
  
  #include "postgres.h"
  

---(end of broadcast)---
TIP 8: explain analyze is your friend


Re: [PATCHES] $PostgreSQL$ header -- Post 7.4 Release

2003-11-10 Thread Neil Conway
Rod Taylor <[EMAIL PROTECTED]> writes:
> In src/backend/utils/adt/inet_net_ntop.c and
> src/backend/utils/adt/inet_net_pton.c it would appear that the $Id$ tag
> is assigned to a variable (rcsid).

... which is never referenced AFAICT, even if the necessary
preprocessor definitions are set so that it is seen by the compiler in
the first place. Is there a reason not to remove this code from both
files?

-Neil


---(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 mailing list cleanly