Hi,
That's a good start. Couple of notes:
DEFINE OSC 4 does not SET the oscillator to 4Mhz it simply tells the compiler at what speed you INTEND to run the chip so it (the compiler) can calculate delay loops etc.
CMCON = 0 does not turn off the comparator, look at the datasheet.

/Henrik.