public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [PATCH, committed, AVR] Change my email address
@ 2009-05-23  7:16 Denis Chertykov
  2009-05-29 20:28 ` Joseph S. Myers
  0 siblings, 1 reply; 3+ messages in thread
From: Denis Chertykov @ 2009-05-23  7:16 UTC (permalink / raw)
  To: gcc-patches

2009-05-23  Denis Chertykov  <chertykov@gmail.com>

	* config/avr/avr.c: Change my email address.
        * config/avr/avr.h: Likewise.
        * config/avr/avr.md: Likewise.
        * config/avr/avr-protos.h: Likewise.
        * config/avr/libgcc.S: Likewise.


===================================================================
--- avr-protos.h	(revision 147816)
+++ avr-protos.h	(working copy)
@@ -2,7 +2,7 @@

    Copyright (C) 2000, 2001, 2002, 2003, 2004, 2006, 2007, 2008, 2009
    Free Software Foundation, Inc.
-   Contributed by Denis Chertykov (denisc@overta.ru)
+   Contributed by Denis Chertykov (chertykov@gmail.com)

    This file is part of GCC.

Index: libgcc.S
===================================================================
--- libgcc.S	(revision 147816)
+++ libgcc.S	(working copy)
@@ -1,7 +1,7 @@
 /*  -*- Mode: Asm -*-  */
 /* Copyright (C) 1998, 1999, 2000, 2007, 2008, 2009
    Free Software Foundation, Inc.
-   Contributed by Denis Chertykov <denisc@overta.ru>
+   Contributed by Denis Chertykov <chertykov@gmail.com>

 This file is free software; you can redistribute it and/or modify it
 under the terms of the GNU General Public License as published by the
Index: avr.md
===================================================================
--- avr.md	(revision 147816)
+++ avr.md	(working copy)
@@ -3,7 +3,7 @@
 ;;   for ATMEL AVR micro controllers.
 ;;   Copyright (C) 1998, 1999, 2000, 2001, 2002, 2004, 2005, 2006, 2007, 2008
 ;;   Free Software Foundation, Inc.
-;;   Contributed by Denis Chertykov (denisc@overta.ru)
+;;   Contributed by Denis Chertykov (chertykov@gmail.com)

 ;; This file is part of GCC.

Index: avr.c
===================================================================
--- avr.c	(revision 147816)
+++ avr.c	(working copy)
@@ -1,7 +1,7 @@
 /* Subroutines for insn-output.c for ATMEL AVR micro controllers
    Copyright (C) 1998, 1999, 2000, 2001, 2002, 2004, 2005, 2006, 2007, 2008,
    2009 Free Software Foundation, Inc.
-   Contributed by Denis Chertykov (denisc@overta.ru)
+   Contributed by Denis Chertykov (chertykov@gmail.com)

    This file is part of GCC.

Index: avr.h
===================================================================
--- avr.h	(revision 147816)
+++ avr.h	(working copy)
@@ -2,7 +2,7 @@
    for ATMEL AVR at90s8515, ATmega103/103L, ATmega603/603L microcontrollers.
    Copyright (C) 1998, 1999, 2000, 2001, 2002, 2003, 2004, 2005, 2006, 2007,
    2008, 2009 Free Software Foundation, Inc.
-   Contributed by Denis Chertykov (denisc@overta.ru)
+   Contributed by Denis Chertykov (chertykov@gmail.com)

 This file is part of GCC.

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH, committed, AVR] Change my email address
  2009-05-23  7:16 [PATCH, committed, AVR] Change my email address Denis Chertykov
@ 2009-05-29 20:28 ` Joseph S. Myers
  2009-05-29 20:42   ` Richard Guenther
  0 siblings, 1 reply; 3+ messages in thread
From: Joseph S. Myers @ 2009-05-29 20:28 UTC (permalink / raw)
  To: Denis Chertykov; +Cc: gcc-patches

On Sat, 23 May 2009, Denis Chertykov wrote:

> 2009-05-23  Denis Chertykov  <chertykov@gmail.com>
> 
> 	* config/avr/avr.c: Change my email address.
>         * config/avr/avr.h: Likewise.
>         * config/avr/avr.md: Likewise.
>         * config/avr/avr-protos.h: Likewise.
>         * config/avr/libgcc.S: Likewise.

Examining source files shows that many if not most email addresses are out 
of date there - that they tend to show the address when a file was 
contributed rather than aiming to show a current one.

I'm inclined to say that source files should not be giving email addresses 
for contributors at all, that MAINTAINERS is the place to keep up-to-date 
addresses and that the ChangeLogs are the place to keep the address 
someone had when contributing a file or a change to a file.

-- 
Joseph S. Myers
joseph@codesourcery.com

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH, committed, AVR] Change my email address
  2009-05-29 20:28 ` Joseph S. Myers
@ 2009-05-29 20:42   ` Richard Guenther
  0 siblings, 0 replies; 3+ messages in thread
From: Richard Guenther @ 2009-05-29 20:42 UTC (permalink / raw)
  To: Joseph S. Myers; +Cc: Denis Chertykov, gcc-patches

On Fri, May 29, 2009 at 10:27 PM, Joseph S. Myers
<joseph@codesourcery.com> wrote:
> On Sat, 23 May 2009, Denis Chertykov wrote:
>
>> 2009-05-23  Denis Chertykov  <chertykov@gmail.com>
>>
>>       * config/avr/avr.c: Change my email address.
>>         * config/avr/avr.h: Likewise.
>>         * config/avr/avr.md: Likewise.
>>         * config/avr/avr-protos.h: Likewise.
>>         * config/avr/libgcc.S: Likewise.
>
> Examining source files shows that many if not most email addresses are out
> of date there - that they tend to show the address when a file was
> contributed rather than aiming to show a current one.
>
> I'm inclined to say that source files should not be giving email addresses
> for contributors at all, that MAINTAINERS is the place to keep up-to-date
> addresses and that the ChangeLogs are the place to keep the address
> someone had when contributing a file or a change to a file.

It's often that the e-mail address at the time of contribution implicitly
mentions the affiliation of the contributor that is often the "real"
contributor of the code.

Richard.

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2009-05-29 20:41 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2009-05-23  7:16 [PATCH, committed, AVR] Change my email address Denis Chertykov
2009-05-29 20:28 ` Joseph S. Myers
2009-05-29 20:42   ` Richard Guenther

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).