Quote Originally Posted by BobS0327
I guess I've been casting malloc for years and just comes as second nature. I would assume this non cast approach is part of the ISO standard?
prelude expalins it better than myself. Although if you compiled it as C++ you would need to cast it. try taking out this replace the first call to malloc with this and compile it
nBinaryTreeNode *CreateBTNode (char *SBinaryTreeData)
    nBinaryTreeNode *nTempNode =  malloc (sizeof (nBinaryTreeNode));
Thanx for keeping me honest.
Thats all the compiler, it doesn't like being lied to.