List:Internals« Previous MessageNext Message »
From:Jim Winstead Date:March 19 2007 11:54pm
Subject:Re: review of patch for Bug#27175: "invalid column attributes in create/alter table"
View as plain text  
On Mon, Mar 19, 2007 at 06:45:47PM -0400, Chad MILLER wrote:
> Our Doxygen guidelines are new and (AFAIK) we don't have any options  
> recommended, and I advocate that we make "JAVADOC_AUTOBRIEF" on, and  
> so this could be
>   /**
>     BitMask of Bits used in "type" (only used in sql_yacc).  Keep  
> track of
>     each bit which has been explicit set to 0 or 1.  This allows to  
> detect
>     conflicts between user options.  Otherwise a user could specify
>     "c int  PRIMARY KEY  NULL".
>   */
> which I think you'll agree is prettier.

+1 to that.

jim
Thread
review of patch for Bug#27175: "invalid column attributes in create/alter table"Chad MILLER19 Mar
  • Re: review of patch for Bug#27175: "invalid column attributes in create/alter table"Jim Winstead19 Mar
    • Re: review of patch for Bug#27175: "invalid column attributes increate/alter table"Jay Pipes20 Mar
      • Re: review of patch for Bug#27175: "invalid column attributes increate/alter table"David Shrewsbury20 Mar
      • "autobrief" in doxygen, was Re: review of patch for Bug#27175: "invalid column attributes in create/alter table"Chad MILLER20 Mar
  • Re: review of patch for Bug#27175: "invalid column attributes increate/alter table"David Shrewsbury20 Mar
  • Re: review of patch for Bug#27175: "invalid column attributes increate/alter table"Martin Friebe20 Mar
Re: review of patch for Bug#27175: "invalid column attributes increate/alter table"Martin Friebe21 Mar