ANSI strings, comment style, ...
Doug Evans
devans@cygnus.com
Tue Nov 4 15:39:00 GMT 1997
I'm working on adding this patch, but a few comments.
You're using ANSI C facilities to stop strings from running past
80 columns. Someone correct me if I'm wrong but I believe
that's a no-no. [Is it written down somewhere what, if any,
of ANSI C facilities one can use for GAS? Should we set a timeframe for
when we'll start allowing such things?]
Also, I couldn't find a passage in the GNU coding conventions that explicitly
says so, but I think it's prefered that one write comments like this:
void
foo ()
{
/* this is variable a, blah blah blah */
int a;
/* this is variable b, blah blah blah */
int b;
/* this is variable c, blah blah blah */
int c;
...
}
rather than this:
void
foo ()
{
int a; /* this is variable a, blah blah blah */
int b; /* this is variable b, blah blah blah */
int c; /* this is variable c, blah blah blah */
...
}
[i.e. write comments on a line by themselves, above what they refer to,
rather than beside]
I've applied your commenting changes as is [the ones that reduce the
column width] but you may wish to switch over instead.
Mon Oct 27 21:44:30 1997 Ian Dall <Ian.Dall@dsto.defence.gov.au>
* write.h (struct fix): comment fx_pcrel_adjust field.
* write.c (print_fixup): Use TC_FIX_DATA_PRINT (if defined) to
print out MD fields of fix.
* frags.c (frag_var, frag_variant): Use TC_FRAG_INIT macro (if
defined) to initialize MD fields in frag.
* doc/internals.texi (Frags): document fr_opcode_frag and
fr_opcode_offset fields. Documetn TC_PCREL_ADJUST macro.
* config/tc-ns32k.h: Add comments. Remove obsolete
BFD_FAST_SECTION_FILL definition, change prototypes for
fix_new_ns32k and fix_new_ns32k_exp to add new arguments
opcode_frag and opcode_offset and remove
pcrel_adjust. (TC_FIX_TYPE): add opcode_fragP and opcode_offset
fields.
(TC_FIX_DATA_PRINT): new macro to print out TC_FIX_TYPE.
(TC_FRAG_INIT): new macro to initialize machine dependent field in
frags.
(frag_opcode_frag, frag_opcode_offset, frag_bsr): macros to access
MD fields in frag structure.
(fix_im_disp, fix_bit_fixP, fix_opcode_frag, fix_opcode_offset,
fix_bsr): macros to access MD fields in fix structure.
* config/tc-ns32k.c: Avoid overlength lines. Align comments. Don't
use struct opcode_location as these fields are now in the frag
structure. (convert_iif): Call frag_more as it is needed instead
of trying to allocate for the whole insn. Avoid call of frag_more
with negative argument. (md_pcrel_adjust, md_fix_pcrel_adjust,
md_apply_fix, md_estimate_size_before_relax, md_pcrel_from,
tc_aout_fix_to_chars): use accessor macros to get md fields in fix
and frag structures. (fix_new_ns32k, fix_new_ns32k_exp): add new
arguments opcode_frag and opcode_offset and remove pcrel_adjust.
(convert_iif, cons_fix_new_ns32k): call fix_new_ns32k,
fix_new_ns32k_exp with changed arguments.
* as.h: Move fragS typdef to before struct frag definition.
Remove ns32k frag substructure and add fr_ns32k substructure.
More information about the Gas2
mailing list