DrewNaylor / UXL-Launcher

An app launcher meant to launch Microsoft Office 2010, 2013, 2016, 2019 desktop software/apps, including ones from Office 365. Not associated with Microsoft.
Apache License 2.0
7 stars 1 forks source link

Simplify the error handling code by moving it all to its own file and using a string to tell the user about anything specific, like the application that we attempted to launch. #77

Closed DrewNaylor closed 7 years ago

DrewNaylor commented 7 years ago

This would be really great and would reduce the size of the application by a large amount. I'm thinking what we could do is, keep the code that does Try, but put everything else in its own Public Shared Sub which then gets referenced from the Try area. Also, right below the Try code, update a string that's created at the launch of the application. That string will be used in the error messages, mainly for the "app not found, change your settings" error messages.

DrewNaylor commented 7 years ago

I believe this is a success:

isolated error handler success

DrewNaylor commented 7 years ago

So I've isolated all the buttons that launch apps and the file size of UXL-Launcher.exe after doing so was 560 KB, down from 568 KB before the error handler isolation. What I did doesn't include any of the other extra error handlers that exist in UXL Launcher.

smaller exe file - copy

DrewNaylor commented 7 years ago

I would have to do a lot of work to the isolated error handlers before placing it into master.

DrewNaylor commented 7 years ago

I actually pulled this into master in May 2017. This means that version 3.1 will benefit from this change.