Since the things that I see have already been mentioned I'll just stick to that one river in the top left. I would not expect a river to arc back through the mountains, valley or not. The land around a mountain chain is all uplifted, and water runs downhill, so I'd expect that river to continue on to the lower left where there is a nice little cove/bay to empty into. Good start, though.