Hello!
I have uVision that compiles fine with the C51 v7.03 compiler and the related package, but not complete with the 8.06. I used two different Keil installations. All files are in the same folder.
In the 8.06 I get linker errors like "object does not fit in to pdata page" and "0080H". This looks like the compiler was thinking the PDATA was only 128 bytes, but it is set to 256 bytes in the startup.a51. Any clue what's different in the newer Keil package?
Also there is a warning in 8.06 (which does not show in 7.03) "converting non-pointer to pointer" on this
ptr_xdata = sPtr_obj->Adresse;
while the vars are set like this:
uchar uc_set_obj( uchar pdata *ptr_Set) { uchar i; uchar xdata *ptr_xdata; struct stOBJADR code *sPtr_obj; sPtr_obj=&Obj[*ptr_Set]; . . . ptr_xdata = sPtr_obj->Adresse; }
The struct stOBJADR has a member "uint Adresse;"
I can see no wrong use of the pointers. I just want to be sure that the warning does not affect the code to not work correctly.
"Guys, please don't be so picky."
This is not picky - you have some very basic misunderstandings here! It's just as well that the compiler did warn you - otherwise you're heading for some really big traps!
"A pointer's address for me is the address it points to."
That is simply incorrect!
A pointer is a "variable" like any other; as such, it has a value, and it has an address - and they are in no way related!
"And this address (sic) is always uint, means two bytes."
That is not true in general, and not true for the 8051 specifically:
A pointer to DATA, IDATA or PDATA space is just a single byte; A pointer to CODE or XDATA is, indeed, 2 bytes; A Generic pointer - able to address any of the above - is three bytes
"So if you're setting a pointer address (sic), you should always use an uint"
Only in cases where the size of a uint (whatever that means) matches the size of the pointer.
"while a char value would also do"
Not necessarily: in general, you can only say that a char value might also do.
"Not how many bytes a pointer actually consumes when compiled to machine code."
But that is the value that actually matters - because that is the value that the code will actually be using!
The address of a pointer (a pointer's address) is a pointer to a pointer, and the concept is called dual indirection.
You need to use dual indirection when you program in C and want a C function to take a pointer as parameter and be able to modify the parameter on return.
It is also a common concept on hardware platforms that does not have a Memory Management Unit (MMU) and where you need to move allocated memory blocks around without having to know what parts of the code that have a reference to the memory blocks.
But no, it isn't picky to very clearly separate the value and the address of a pointer. There are a large number of programming situations where you must keep track of the number of indirections - and the compiler do keep track of the data type of pointers, i.e. a pointer to an int or a pointer to a pointer to an int are two very different data types, resulting in a warning if you mismatch your assign. The indirection of a pointer to a pointer to int is a pointer to int. The indirection of a pointer to int is the int, i.e. no longer an address.
You are all right, but I think this code is still not generally wrong. It wasn't written by me and it always worked. In order to distinguish the addresses a pointer is related to, I call the address the pointer points to the pointer address. The address where the pointer itself is located or managed, I call base address. But this address is commonly not used in consideration. When I'm writing C, then it's painful enough to fulfill the microprocessor's picky demands, so I don't want to care about if the pointer's base address consists of two or three bytes. Only if I wouldn't have enough RAM.
Back to my problem: Putting a cast in front of the particular line did not succeed. It still reports warning C289, which is not described in the C51 help file.
"I call the address the pointer points to the pointer address."
In your own little world, you can call it what you like. You just have to understand that it is completely contrary to what the rest of the world understands by that term. It also suggests that you have not fully grasped what's going on.
"The address where the pointer itself is located or managed, I call base address"
That is also contrary to the generally-accepted meaning of that term. A "base" address is generally accepted to mean the datum (or reference) from which an offset or relative address is calculated.
" But this address is commonly not used in consideration"
Oh yes it is!
As Per said, having a pointer to a pointer is quite common; in fact, the C51 compiler supports up to 19 levels of indirection! http://www.keil.com/support/man/docs/c51/c51_xd.htm
It is also common to need to know the address (ie, location) of a pointer when debugging...
"When I'm writing C, then it's painful enough to fulfill the microprocessor's picky demands"
Yes - you can get away with sloppy terminology when talking to other people, but not when talking to a compiler!
If you think 'C' is picky, you should try writing in Pascal or Ada...
Assembler may seem less "picky", but that just puts even more responsibility on the programmer to ensure that they have everything right!
"I don't want to care about if the pointer's base address (sic) consists of two or three bytes"
In general, you don't need to - it's only when you start trying to assign a non-pointer value (such as a uint) to a pointer that you need to be really sure of what you are doing...
"Only if I wouldn't have enough RAM."
No, that has nothing to do with it! If you mess up a pointer, it can point to absolutely anywhere in memory, and do untold damage.
That's why pointers in 'C' are both so powerful and so dangerous!
Which is why it is so important that you have a really firm understanding of what's going on.
Which is why compilers tend to give warnings when there's any doubt...
"Back to my problem: Putting a cast in front of the particular line did not succeed"
Show exactly what cast you used!
But you do have to learn to start thinking and talking about a pointers value, not a pointers address.
You will get into trouble if you do not see it as a variable. And variables have values. Sometimes they also have memory addresses.
A base address is something completely different, so please do not continue thinking about the address of a pointer as a base address. The start address of an array could be seen as a base pointer but a pointer and an array are not identical.
char buf[1000]; char* p = buf;
Now you think that the address of p is a base address. Why? The address of p has nothing to do with buf, and isn't the base of anything meaningful.
The reason it is painful for you to "fulfill the microprocessor's picky demans" is that you have not spent any time on figuring out why. If you know why something is as it is, you will start to see a pattern. And when you see the pattern, it will not feel like any picky demands anymore. People have spent huge amounts of time thinking about logical (not picky) solutions to the language design, the processor instruction design, the processor peripherials etc.
Everything you choose to treat as black magic will continue to be black magic. And black magic is something we humans are not comfortable with. It is up to you to decide if you want to take control, or if you cant to continue to be confused. As long as you decide to invent your own terminology, you will never be able to buy books and read. You will just be confused, since the books will - in your mind - be picky and invent the wrong terminology...
You say you don't want to have to care about if a pointer's base address (sic!) is two or three bytes. You don't have to know the size of a pointer variable, as long as you stop playing with addresses in integers. For small chips it may still be good to know the size of a pointer variable, since you have limited memory in the chip. But that is a completely different issue. Right now, your problem is that you are writing code where you need to make a lot of assumptions. This specifically forces you to have the knowledge to be able to figure out if the assumptions are correct or not.
You still have a warning?
Maybe you should post the full set of data declarations, and your rewritten source line - including the type cast. Only then will people on this forum be able to tell if your cast is correct or not.
LOL.
You won Andy. But there are a lot of similarities in our answers ;)
So if you're setting a pointer address, you should always use an uint value
Not true. intptr_t is the integer type wide enough to hold a pointer; ptrdiff_t is the type wide enough to hold a difference of two pointers. Neither one is necessarily the same as an unsigned int.
Compilers are allowed to warn about whatever they feel like. Given the complexity of the 8051 addresses spaces, as mentioned above, along with the variable width of pointers in the C51 implementation, I think it's a good thing for C51 to be sensitive in this regard. You're always free to disable or ignore the warning if you think it's silly, but in this case I'd recommended taking it to heart and putting in the proper casts where needed to convert the data type explicitly, rather than relying on default, implicit, conversions -- even if those did work in version 7.
char buf[1000]; char* p = buf; Now you think that the address of p is a base address. Why? The address of p has nothing to do with buf, and isn't the base of anything meaningful. This is what I consider as pointer address.
But we better stop here. I'm programming since a few years now and never had such problems. I think I don't need to read a bible about pointers, just to write some code. That's not the meaning of a computer language. To be more complicated than its inventor.
My conclusion is: if I set a pointer address in XDATA with a number I read from a struct and as long as this number is 2 bytes long and thus valid, then the pointer has to be set. Period. By assigning a pointer's address I'm not doing a pointer conversion, because the pointer only changes the byte it points to. Nothing more, nothing less.
This is what I consider as pointer address.
We know that by now. And it's still wrong, because this use of the term is guaranteed to confuse everyone exposed to it --- apparently including it's original user, i.e. you.
long as this number is 2 bytes long and thus valid,
That "thus" exposes that you still haven't understood what the issue actually is. No, a 2-byte number is not self-evidently "valid" to hold a pointer. You need 3 bytes to hold an arbitrary pointer in C51.
To put it bluntly: you're deluding yourself. You don't understand pointers anywhere as well as you think you do. Your insistence on ill-fitting terminology may well be part of the problem, preventing true understanding.
then the pointer has to be set.
What exactly made you think it wasn't? You say you've been programming for years, but you still don't recognize the difference between an error and a warning?
"I think I don't need to read a bible about pointers"
The way you write about them suggests otherwise.
"as long as this number is 2 bytes long and thus valid"
That is a non-sequitur; ie, the conclusion does not follow from the reason stated.
A 2-byte number might happen to be valid as an XDATA pointer, or it might not - the fact of its size being 2 bytes has absolutely no relation whatsoever to its validity (or otherwise) as a pointer!
"then the pointer has to be set"
There never was any doubt that the pointer would be set - the question is whether it gets set to anything meaningful...
"By assigning a pointer's address I'm not doing a pointer conversion"
By assinging from an int to a pointer you are doing an implicit conversion!
"the pointer only changes the byte it points to"
Eh??! A pointer of itself changes nothing; only when you dereference the pointer does the pointed-to location get accessed.
You still haven't shown the cast that you used and that still gave the warning. If you want to get back to that, then do it...
"That's not the meaning of a computer language. To be more complicated than its inventor."
It isn't complicated; it is really quite simple. You are the one that is complicating it by messing-up the terminology!
Every program "object" - variables and functions - has an address, and that address is the location at which the object itself is stored. Simple. No exceptions.
This is the terminology used by the inventors of the language - Messrs Kernighan & Ritchie.
You are the one who is complicating this by using "address of" when applied to a pointer to mean something different from "address of" when applied to any other object!
Because you have thus modified the meaning of "address of" when applied to a pointer, you then also have to invent another term to use when you do want to refer to the location of the pointer itself!
All this is further complicated by the fact that the non-standard terms you have chosen already have well-established and quite different meanings in the more general context!
Note that all variables are not guaranteed to have a memory address - they could be register-based too. On one hand, that regsiter could be seen as an address, but on the other hand, the compiler could move the variable around if it feels like it. For some processors with few GP registers and slow memory access (at least old 16-bit x86 processors) there was some advantages to switching registers.
Seems like we talking past each other.
In other words: I know what a pointer is, but for me the only thing of interest is what it does. That's what I was talking about. An example:
char *ptr; char a; ptr = 7;
is the same principle as
ptr = &a;
where the address of 'a' is not determined. I'm setting the pointer to point to an address, whether this is a number (an absolute address) or a variable. This is all what is done in the code above. Where is the conversion? I can't see it. Because warning C289 says "converting non-pointer to pointer" in v8.06. Actually I'm setting a pointer to an address of a variable.
1. A struct is declared. 2. The struct has a member 'Adresse', uint type. 3. This member is set to the address of a variable, let's say &test. 4. A struct pointer is set to this struct. 5. A char pointer is declared. 6. The struct pointer is set to point to the member 'Adresse'. 7. The value of 'Adresse' is read out and used to set the char pointer to point to 'test'.
Totally OK for me. Where's the problem?
Where is the conversion? I can't see it.
You cannot see it, but the compiler can. Even though C is a fairly weakly-typed language, an integer and a pointer are not the same for the compiler, and need to be treated differently in a number of situations.
Consider the following code snippet:
unsigned int some_int = 0; unsigned int *some_ptr = 0; some_int++; some_ptr++;
Can you tell what the value of some_int will be after the snippet ? (that's trivial, right ?)
Can you tell what the value of some_ptr will be at the end of the code snippet ? (If yes, please state what it will be and why. If no, please state what information you are missing.)
And that is just one example.
Since C is fairly weakly-typed, the compiler might allow a number of so-called implicit type casts. More high-level languages are usually more strongly-typed, and would require an explicit cast in many situations.
Now back to your warning - can you post the line with the explicity type cast you use that still results in a warning ?
The & in 'C' is the "address-of" operator; the result of the expression
&a
gives the "address-of" a - so its type is self-evidently "pointer to a"
So, when you write
p = &a
the compiler can clearly see that you are assigning a pointer-type expression (&a) to a pointer-type variable (p)
However, when you write
p = 7
it is not self-evident that 7 is intended to be a valid address; more particularly, when you write
p = a
there must be some doubt that you might have meant
hence the compiler gives the warning.
To avoid the warning, you can make you intention clear by using an explicit cast:
p = (char*)a
If you're still getting the warning, you must have done it wrong. If you want an explanation of what you've done wrong, you will have to show what you have done!
The problem is that it is not self-evidently correct.
The compiler has noticed the possibility that what you wrote was not what you intended - and so it gives you a Warning.
The Warning is particularly justified as this is such a common mistake - like writing "=" instead of "==" in an 'if' clause...
One more time: you can stop the warning by using an appropriate explicit type-cast.