zsh-workers
 help / color / mirror / code / Atom feed
* Please, do NOT use GCC as development compiler (or, at least, turn on some conformance checking).
@ 2000-03-03  9:21 Andrej Borsenkow
  0 siblings, 0 replies; only message in thread
From: Andrej Borsenkow @ 2000-03-03  9:21 UTC (permalink / raw)
  To: ZSH workers mailing list

parse.c  2426: [error]:   CFE1137 expression must be a modifiable lvalue
        fdflags(pre) = (map ? FDF_MAP : 0) | other;
        ^

parse.c  2427: [error]:   CFE1137 expression must be a modifiable lvalue
        fdsetother(pre, tlen);
        ^

parse.c  2427: [error]:   CFE1137 expression must be a modifiable lvalue
        fdsetother(pre, tlen);
        ^

parse.c  2427: [error]:   CFE1137 expression must be a modifiable lvalue
        fdsetother(pre, tlen);


That all originates from fdbyte() macro, that looks like:

#define fdbyte(f, i)     ((wordcode) (((unsigned char *) (((Wordcode) (f)) +
1))[i]))

You take a _char_ value and cast it to wordcode == int. This_is_not_lvalue
and can not be assigned to (most probable implementation is temporary
storage in stack or register). I have no idea, what GCC does with it.

So, the simplest way is to provide two sets of macros - GET and SET. And,
BTW, why are they in lower case? I'd prefer such macros in upper case - as
is customary in C ...

cheers

/andrej


^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2000-03-03  9:22 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2000-03-03  9:21 Please, do NOT use GCC as development compiler (or, at least, turn on some conformance checking) Andrej Borsenkow

Code repositories for project(s) associated with this public inbox

	https://git.vuxu.org/mirror/zsh/

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).