This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH v3] sim: bfin: new port


On Monday 21 February 2011 16:53:45, Mike Frysinger wrote:
> On Monday, February 21, 2011 04:54:36 Joel Brobecker wrote:
> > Question: Why do you have .h files whose name starts with an underscore?
> > For instance: sim/bfin/_proc_list.h...
> 
> it's meant to convey that the header isnt meant to be generally included by 
> random files.  they can only be included in specific instances since they're 
> simple lists which need certain preprocessor directives in place before they 
> can be included.  i'm trying to avoid bit rot related to copying & pasting the 
> same list of numbers over and over in multiple places.

That's a good idea.  I notice a copyright header is missing though.
FYI, on the gdb/ side, we name those files "*.def".
E.g., ada-operator.def, common/ax.def, std-operator.def.

-- 
Pedro Alves


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]