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: [python][patch] And range method to type


This is not an official review, as I don't have much experience
exporting to Python, but it seems right to me.  Just one minor
cosmetic comment:

> +  if (TYPE_CODE (type) != TYPE_CODE_ARRAY &&
> +      TYPE_CODE (type) != TYPE_CODE_STRING &&
> +      TYPE_CODE (type) != TYPE_CODE_RANGE)

The && operators should be at the beginning of the line (GNU Coding
Standards):

  if (TYPE_CODE (type) != TYPE_CODE_ARRAY
      && TYPE_CODE (type) != TYPE_CODE_STRING
      && TYPE_CODE (type) != TYPE_CODE_RANGE)

-- 
Joel


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