screwed up programmer

[Bill Cunningham]

Can anyone tell me what I'm doing here.

#include <stdio.h>
#include "p.h"

extern int show(const char *p){
printf(%c,p);}

Compile into p.o

p.h says
#include <stdio.h>
int show(const char *p);

prog.c says

#include <stdio.h>
#include "p.h"

int main(){
show("hello");}
compile gcc prog.c -o prog

This is all f*cked up. What's wrong with that printf? Go ahead and laugh
and throw tomatoes.

Bill



-----= Posted via Newsfeeds.Com, Uncensored Usenet News =-----
http://www.newsfeeds.com - The #1 Newsgroup Service in the World!
-----== Over 100,000 Newsgroups - 19 Different Servers! =-----

[Jack Klein]

On Sat, 4 Oct 2003 01:06:42 -0400, "Bill Cunningham" <no****@net.net >
wrote in comp.lang.c:

Can anyone tell me what I'm doing here.

#include <stdio.h>
#include "p.h"

extern int show(const char *p){
printf(%c,p);}
The call to printf() above is certainly wrong. The first argument to
printf() must be a string, perhaps you meant "%c"?

If that's the case the second argument must be a char, which p is not.
p is a pointer to char, which happens to point to the string literal
"hello".

If you wanted to print the first character of that string, you need to
write:

printf("%c", *p);

....*p is a single char, what "%c" needs.

Two other things about your function show...

1. Putting extern on a function declaration, prototype, or definition
is unnecessary and redundant. Unless it contains the static keyword,
it has external linkage.

2. You state that the function will return an int, then you don't
return anything.
Compile into p.o

p.h says
#include <stdio.h>
int show(const char *p);

prog.c says

#include <stdio.h>
#include "p.h"

int main(){
show("hello");}
compile gcc prog.c -o prog

This is all f*cked up. What's wrong with that printf? Go ahead and laugh
and throw tomatoes.

Bill

--
Jack Klein
Home: http://JK-Technology.Com
FAQs for
comp.lang.c http://www.eskimo.com/~scs/C-faq/top.html
comp.lang.c++ http://www.parashift.com/c++-faq-lite/
alt.comp.lang.l earn.c-c++ ftp://snurse-l.org/pub/acllc-c++/faq

[Richard Heathfield]

Bill Cunningham wrote:

Can anyone tell me what I'm doing here.

#include <stdio.h>
#include "p.h" extern int show(const char *p){
printf(%c,p);}

Syntax error. printf requires a format string, and %c isn't one. "%c", on
the other hand, is one. Your compiler should have told you this. Note that
"%c" would match with a character, not a character string. Perhaps you mean
"%s", or perhaps you mean *p - it's impossible to tell from your code.

--
Richard Heathfield : bi****@eton.pow ernet.co.uk
"Usenet is a strange place." - Dennis M Ritchie, 29 July 1999.
C FAQ: http://www.eskimo.com/~scs/C-faq/top.html
K&R answers, C books, etc: http://users.powernet.co.uk/eton

[Bill Cunningham]

Note that

"%c" would match with a character, not a character string. Perhaps you mean "%s", or perhaps you mean *p - it's impossible to tell from your code.

So these 2 prototypes would be alright?

printf("%c",*p) ;
or printf("%s",p);
casting and precedence is my worst enemy in C. When to cast and I seen code
with bunches of parenthesis which is confusing to the beginner. Me anyway.

Bill

-----= Posted via Newsfeeds.Com, Uncensored Usenet News =-----
http://www.newsfeeds.com - The #1 Newsgroup Service in the World!
-----== Over 100,000 Newsgroups - 19 Different Servers! =-----

[Serve La]

"Bill Cunningham" <no****@net.net > wrote in message
news:3f******** **@corp.newsgro ups.com...

Note that

"%c" would match with a character, not a character string. Perhaps you

mean

"%s", or perhaps you mean *p - it's impossible to tell from your code.

So these 2 prototypes would be alright?

printf("%c",*p) ;
or printf("%s",p);

Yes, the first prints the first char of where p is pointing to, the second
prints all characters until a '\0' is encountered.

[Richard Heathfield]

Bill Cunningham wrote:

Note that

"%c" would match with a character, not a character string. Perhaps you mean

"%s", or perhaps you mean *p - it's impossible to tell from your code.

So these 2 prototypes would be alright?

printf("%c",*p) ;
or printf("%s",p);

Those are not prototypes. They are function calls. The first one prints the
first character in the string to which p points. The second prints the
whole string to which p points.
casting and precedence is my worst enemy in C.
I see no casts here, nor do I see any need for them.
When to cast and I seen
code with bunches of parenthesis which is confusing to the beginner. Me
anyway.

Casting is usually wrong, and the places where it's right aren't the places
you'd expect. In good C code, casts are very rare.

--
Richard Heathfield : bi****@eton.pow ernet.co.uk
"Usenet is a strange place." - Dennis M Ritchie, 29 July 1999.
C FAQ: http://www.eskimo.com/~scs/C-faq/top.html
K&R answers, C books, etc: http://users.powernet.co.uk/eton