This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC 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: [RFC] patch: libffi dejanu test framework


Andreas Tobler <toa@pop.agri.ch> writes:

> Is it ok to add this type of header to each testcase (tc):
>
> /* File:	    cls_struct_2byte
>   * Area:	    ffi_call / closure_call
>   * Description:     check the structure passing/calling for small structs.
>   * 		    Small structs may fit into one register depending on
>   *		    the ABI.
>   * Restrictions:    none.
>   * PR:		    none.
>   * Contributed by:  <a.tobler@schweiz.ch> 27082003
>   */
>
> This gives us some information on how, who, why and when.

Yes, I think that's a good idea.  Our coding style prefers not to have
a column of asterisks, but this is a minor detail.

It is probably unnecessary to name the file in the header.

zw


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