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: [RFC] msp430 instruction decoder


Hi Mike,

This patch was created by DJ, but he is on vacation so Kevin was asked to submit it on his behalf. Now I am wading in to address the points that you raised...
typically it's better to omit generated files from patches posted to the list.
it's often times a lot of noise (like in this case).

Agreed - I have done so in the revised patch attached to this email.

missing standard copyright/license block

Added.

+typedef struct
+{
+  MSP430_Opcode_Decoded * msp430;

shouldn't the style be:
	MSP430_Opcode_Decoded *msp430;


seems like general style guidelines are violated a lot in this file.  do we
normally ignore that in the opcodes/ tree ?

No, and I have revised the formatting in the patch files. They do now conform to the GNU Coding Standard. Note - the msp430-decode.opc file includes some comments that do not quite conform to the standard. This is because these comments are directives to the opc2c program and need to remain in their current format.

Do you have any more concerns with this patch ?

Cheers
  Nick

opcodes/ChangeLog
	* msp430-decode.opc: New.
	* msp430-decode.c: New/generated.
	* Makefile.am (TARGET_LIBOPCODES_CFILES): Add msp430-decode.c.
	(MAINTAINER_CLEANFILES): Likewise.
	Add rule to build msp430-decode.c frommsp430decode.opc 		
        using the opc2c program.
	* Makefile.in: Regenerate.
	* configure.in: Add msp430-decode.lo to msp430 architecture files.
	* configure: Regenerate.

include/opcode/ChangeLog
	* msp430-decode.h: New.


Attachment: msp430-opcodes-dis.patch.bz2
Description: application/bzip


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