Warning: Spoiler! (Click to show)
Since you asked...
Sorry, but I have to use JC for illustration purposes. Put your thumb over his face if you feel awkward about it. I have highlighted the problems in red and the good stuff or solutions in green.
JC already knows what I'm about to tell you and I'm not revealing and secrets here, just commonly held knowledge about not looking like a hobbit when you wear clothes. The fact that he persists in wearing these mistakes tells me that he is either a masochist, half blind, or purposefully belligerent. Possibly all three. First the shoulders: You can see the slope of the shoulders, hi lighted in red, is rather steep. For the person of average, or above average, height this is desirable. However, for the short man this enhances short stature. A slightly more construction, hi lighted in green, would give an illusion of more height by raising the height of the shoulder. Criticizing someone's large lapel width is like making a joke about his mother on SF, but whatevs. By slightly slimming the lapel JC would underplay the lateral lines of the lapel, creating a more vertical line and giving yet another accentuation of height. The buttoning point is much too high for a short person. Drop it lower and give yourself a vertical plane to mirror the line of the legs. Patch pockets are a big no-no short short people. They create not only more lateral lines on the torso, but literally putting a stubby block of fabric in the worst possible place, the torso. But you have to have pockets, right? The solution is to be as discrete with your pockets as possible. Flapped pockets are ok, but besom is better. Jacket length is easy. Longer jacket lengths reduce the appearance of length in the legs so short dudes should avoid them at all costs. They need a jacket as short as possible while still conforming to classic tastes. Not only will this extend the line of the legs, but it will make the torso appear longer. Another big no-no for the shorties, trouser cuffs. Yet again, lateral lines breaking up the length of the legs. Keep them trews uncuffed. JC's trouser length is really good. Short-stops never want to have too much break, if any at all. More lateral lines, again. Not a big deal for obviously tall or average height people, but if you are really short you want to use every device you can to create the illusion of height.
Now you may be saying "But if I can't have wide lapels, patch pockets, and cuffed trousers how can I be a SF star???" The answer is to damn the torpedoes and do what is right for your stature.
Follows these guidelines and you'll look like Andre The Giant. Well, not really, but at least you won't look like Frodo.
This critique is spot-on, but I'd be feeling a bit crushed if I were JCM reading it. And if he has just read it, well - I'm sure there are still many people who like that suit despite the faults Caustic has just pointed out. Mainly, it's beautifully made from nice cloth and it fits well.
But yes, I find myself unable to disagree with just about every stylistic point Caustic has made above.