I don't see anything obvious, but I would roll through your code and ask yourself if every last piece on that circuit is a necessity. I don't personally see why you need the LED and diodes, but I don't know what your trying to rock on the other side of that pic.
the led should light when you have power, I like that, keep it. Use a meter or a scope to measure the volatage on the pic pins esp. power and make sure they are correct. Does the pic have a clock signal ( look with a scope ). If you are on a bread board using a crystal do not go above 4 meg hz unless you do not mind fiddling around to make sure the clock is running.
Use a program so dog simple that you are sure it is ok.