I'd echo what has been said by others. You have nailed it. If you want nit picking then as already mentioned:

Sort your laces out - I like the bow on the outside shows a bit of style.

I would wear the pin lower but....

Leave the sporran where it is. It should be protecting the family assets not covering your belly button.

The bottom button of your waistcoat is said to be related to King Edward being too fat to do his up and everyone else following suit so as not to be rude - but there are other versions see;

http://wiki.answers.com/Q/Why_doesn%27t_one_button_the_bottom_button_on_a_su it_or_waistcoat and

http://en.wikipedia.org/wiki/Waistcoat

Some Scots will deliberately do up their bottom buttons as they feel that leaving it undone is an English affectation and they wish to be rude to the (dead) King. I would not agree with this but...