>+ // This empty catch block needs to be here because, although
>+ // msgHdr.folder will be null for .eml files, it will still throw an
>+ // exception when you try to access it.
>+ }
>+ return null;
But please just do this functionality inline instead of as a function. (And i think it should just say that it throws, not that it's null.)
(From update of attachment 379741) msgHdr) folder. server. type;
>+function getServerType(
>+{
>+ try {
>+ return msgHdr.
>+ } catch (ex) {
catch on a new line
>+ // This empty catch block needs to be here because, although
>+ // msgHdr.folder will be null for .eml files, it will still throw an
>+ // exception when you try to access it.
>+ }
>+ return null;
But please just do this functionality inline instead of as a function. (And i think it should just say that it throws, not that it's null.)
Otherwise looks good to me.