Re: [PATCH] include/libbb.h: declare messages with ALIGN1

2012-07-31 Thread Aurelien Jarno
On Tue, Jul 24, 2012 at 04:30:27PM +0200, Denys Vlasenko wrote: > On Mon, Jul 16, 2012 at 10:56 AM, Aurelien Jarno wrote: > > Some messages strings are defined with ALIGN1 in libbb/messages.c > > to make sure strings are not aligned and thus to save some bytes. The > > corresponding declaration in

Re: [PATCH] include/libbb.h: declare messages with ALIGN1

2012-07-24 Thread Denys Vlasenko
On Mon, Jul 16, 2012 at 10:56 AM, Aurelien Jarno wrote: > Some messages strings are defined with ALIGN1 in libbb/messages.c > to make sure strings are not aligned and thus to save some bytes. The > corresponding declaration in include/libbb.h should also use ALIGN1, > otherwise the compiler may as

Re: [PATCH] include/libbb.h: declare messages with ALIGN1

2012-07-17 Thread Aurelien Jarno
On Mon, Jul 16, 2012 at 06:54:38PM +0400, Michael Tokarev wrote: > On 16.07.2012 12:56, Aurelien Jarno wrote: > > Some messages strings are defined with ALIGN1 in libbb/messages.c > > to make sure strings are not aligned and thus to save some bytes. The > > corresponding declaration in include/libb

Re: [PATCH] include/libbb.h: declare messages with ALIGN1

2012-07-16 Thread Michael Tokarev
On 16.07.2012 12:56, Aurelien Jarno wrote: > Some messages strings are defined with ALIGN1 in libbb/messages.c > to make sure strings are not aligned and thus to save some bytes. The > corresponding declaration in include/libbb.h should also use ALIGN1, > otherwise the compiler may assume they are

[PATCH] include/libbb.h: declare messages with ALIGN1

2012-07-16 Thread Aurelien Jarno
Some messages strings are defined with ALIGN1 in libbb/messages.c to make sure strings are not aligned and thus to save some bytes. The corresponding declaration in include/libbb.h should also use ALIGN1, otherwise the compiler may assume they are aligned and generate wrong code to access them. Thi