Juergen Vigna <[EMAIL PROTECTED]> writes:
| Assert(s); // we don't allow null pointers
Excatly what the standard says.
| But then something like this is not possible:
|
| string str = ptr; // ptr could also be 0
yes, and this should fail.
| but we always have to do something like:
|
| string str;
|
| if (ptr) str = ptr;
Yes.
| I think this should be removed and there should be a check in the constructor
| which just does not initialize if the pointer is a 0 pointer!
NO!! You have to check the standard before doing _ANY_ changes to
lyxstring.
|
| I'll fix this now and if you don't like it then you (you is general not JMarc;)
| can refix this and also all instances in the code where we allocate a string
| like this with the warning never do this :)
I will just put the Assert back in and remove other changes you have
made to lyxstring.
Lgb