- No Android code style. In fact, it's much easier to deal with
left,right, andkeyin the Eclipse debugger instead ofmLeft,mRight, andmKey. - The node names
pandqare left, and I explained why. - The node name is
parentrather thanrootin the methodinsert(Node parent, int key). - Duplicates are not allowed. I'd handle duplicates this waythis way if I needed to. Let me insert the quote I mean:
replaced http://codereview.stackexchange.com/ with https://codereview.stackexchange.com/