This is the mail archive of the ecos-patches@sourceware.org mailing list for the eCos 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: romfs fix take #2


On Nov 14, 2007 3:09 PM, Jonathan Larmour <jifl@ecoscentric.com> wrote:
> Øyvind Harboe wrote:
> >
> >  append result "/* This is a generated file. Do not edit. */\n\n"
> > -append result "static const unsigned char filedata\[\] = {\n"
> > +append result "static CYGBLD_ATTRIB_ALIGN(4) const unsigned char filedata\[\] = {\n"
>
> Thanks. Patch duly applied, although I'm a little uneasy about hard-coding
> 4. It's probably pedantic to make a fuss about it though so I've kept it.

Agreed. However, this script is duplicated and specific to romfs where
4 is poured in concrete anyway....

I'd rather worry about the other places where that tcl script is used...



-- 
Øyvind Harboe
http://www.zylin.com - eCos ARM & FPGA  developer kit


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