what's wrong with the macro

[DaVinci]

#include<stdio. h>
#include<stdlib .h>
#include<string .h>

#define SECONDS_PER_YEA R (60*60*24*365)U L
int main()
{
unsigned long int u_i = SECONDS_PER_YEA R;
printf("%lu\n", u_i);

return 0;
}
cc -g -c -Wall -std=c99 -DDEBUG define.c
define.c: In function 'main':
define.c:17: error: expected ',' or ';' before 'UL'
make: *** [define.o] error 1

[Malcolm]

The qualifer must follow the digits directly.

--
www.personal.leeds.ac.uk/~bgy1mm
freeware games to download.

"DaVinci" <ap***********@ gmail.comwrote in message
news:11******** *************@m 73g2000cwd.goog legroups.com...

>
#include<stdio. h>
#include<stdlib .h>
#include<string .h>

#define SECONDS_PER_YEA R (60*60*24*365)U L
int main()
{
unsigned long int u_i = SECONDS_PER_YEA R;
printf("%lu\n", u_i);

return 0;
}
cc -g -c -Wall -std=c99 -DDEBUG define.c
define.c: In function 'main':
define.c:17: error: expected ',' or ';' before 'UL'
make: *** [define.o] error 1

[Frederick Gotham]

Malcolm posted:

The qualifer must follow the digits directly.

It can be changed to simply:

#define SECONDS_PER_YEA R(60LU*60*24*36 5)

--

Frederick Gotham

[Giorgio Silvestri]

"Frederick Gotham" <fg*******@SPAM .comha scritto nel messaggio
news:Vf******** ***********@new s.indigo.ie...

Malcolm posted:

The qualifer must follow the digits directly.

It can be changed to simply:

#define SECONDS_PER_YEA R(60LU*60*24*36 5)

A space is necessary

#define SECONDS_PER_YEA R (60LU*60*24*365 )
--
Giorgio Silvestri
DSP/Embedded/Real Time OS Software Engineer

[Andrew Poelstra]

On Sun, 2006-10-22 at 00:03 -0700, DaVinci wrote:

#include<stdio. h>
#include<stdlib .h>
#include<string .h>

#define SECONDS_PER_YEA R (60*60*24*365)U L
int main()
{
unsigned long int u_i = SECONDS_PER_YEA R;
printf("%lu\n", u_i);

return 0;
}
cc -g -c -Wall -std=c99 -DDEBUG define.c
define.c: In function 'main':
define.c:17: error: expected ',' or ';' before 'UL'
make: *** [define.o] error 1

You are attempting to compile a file called define.c, which has a syntax
error on line 17. If you want help, you'll have to post the code.
--
Andrew Poelstra <http://www.wpsoftware. net/projects/>

[Richard Heathfield]

DaVinci said:

>
#include<stdio. h>
#include<stdlib .h>
#include<string .h>

#define SECONDS_PER_YEA R (60*60*24*365)U L

It doesn't work like that, I'm afraid. You can, however, do either this:

#define SECONDS_PER_YEA R 31536000UL

or this:

#define SECONDS_PER_YEA R (unsigned long)(60UL * 60UL * 24UL * 365UL)

or something along similar lines.

(Incidentally, your calculation ignores leap years.)

--
Richard Heathfield
"Usenet is a strange place" - dmr 29/7/1999
http://www.cpax.org.uk
email: rjh at above domain (but drop the www, obviously)

[Frederick Gotham]

Richard Heathfield posted:

#define SECONDS_PER_YEA R (unsigned long)(60UL * 60UL * 24UL * 365UL)

Something that came to mind recently...

Do you think one would be right to _always_ enclose macros in parentheses,
irrespective of the fact that it might be impossible to yield to
inappropriate parse? (What I mean by inappropriate parse is the following:

#define DOUBLE(x) (x)+(x)

int i = 5 * DOUBLE(3) * 2; /* Wups! */

As the operator precendence table currently stands (the metaphysical one,
that is :P), there's no problem with your macro above... but should we make
allowances for future additions to the language. It's not impossible that a
new operator would be added which could possibly result in an inappropriate
parse after the macro has been substituted.

Just a thought!

--

Frederick Gotham

[Keith Thompson]

Richard Heathfield <in*****@invali d.invalidwrites :

DaVinci said:

>#include<stdio .h>
#include<stdli b.h>
#include<strin g.h>

#define SECONDS_PER_YEA R (60*60*24*365)U L

It doesn't work like that, I'm afraid. You can, however, do either this:

#define SECONDS_PER_YEA R 31536000UL

or this:

#define SECONDS_PER_YEA R (unsigned long)(60UL * 60UL * 24UL * 365UL)

or something along similar lines.

(Incidentally, your calculation ignores leap years.)

You want parentheses around the second defintion:

#define SECONDS_PER_YEA R ((unsigned long)(60UL * 60UL * 24UL * 365UL)(

Consider "sizeof SECONDS_PER_YEA R".

--
Keith Thompson (The_Other_Keit h) ks***@mib.org <http://www.ghoti.net/~kst>
San Diego Supercomputer Center <* <http://users.sdsc.edu/~kst>
We must do something. This is something. Therefore, we must do this.

[Giorgio Silvestri]

"Keith Thompson" <ks***@mib.orgh a scritto nel messaggio
news:ln******** ****@nuthaus.mi b.org...

Richard Heathfield <in*****@invali d.invalidwrites :

DaVinci said:

#include<stdio. h>
#include<stdlib .h>
#include<string .h>

#define SECONDS_PER_YEA R (60*60*24*365)U L

It doesn't work like that, I'm afraid. You can, however, do either this:

#define SECONDS_PER_YEA R 31536000UL

or this:

#define SECONDS_PER_YEA R (unsigned long)(60UL * 60UL * 24UL * 365UL)

or something along similar lines.

(Incidentally, your calculation ignores leap years.)

You want parentheses around the second defintion:

#define SECONDS_PER_YEA R ((unsigned long)(60UL * 60UL * 24UL * 365UL)(

Probably:

#define SECONDS_PER_YEA R ((unsigned long)(60UL * 60UL * 24UL * 365UL))

is better :-)

>
Consider "sizeof SECONDS_PER_YEA R".

--
Giorgio Silvestri
DSP/Embedded/Real Time OS Software Engineer

[Frederick Gotham]

Keith Thompson posted:

You want parentheses around the second defintion:

#define SECONDS_PER_YEA R ((unsigned long)(60UL * 60UL * 24UL * 365UL)(

Consider "sizeof SECONDS_PER_YEA R".

I don't understand... what could that have parsed to?

--

Frederick Gotham