Bug in Firmware Loader code?

[IndianCheese]

I was sifting through the source code of MPH's firmware loader and found what appears to be an error causing video to malfunction in 2.50. Look at lines 405-406 in the main.c file:

 // For video 2.50
//LoadStartModule("mphfl0:/kd/mgvideo.prx",1);

It appears that that module loading is marked as a comment by accident, and should look like this:

 // For video 2.50
 loadStartModule("mphfl0:/kd/mgvideo.prx",1);

Was this marked off on purpose to prevent execution, or is it just a bug in the code? I would try it, but I am in fear of bricking. I'm not very confident when it comes to unmounting/remounting virtual partitions...

[b8a]

If you're looking for some real answers on this, you should probably post on the forums over at <a href="http://mphwebsite.tuxfamily.org/" target="_blank">MPH's official website</a>.

Just keep in mind that it's not nessecarily a mistake. Code is commented out for any number of reasons, and given the time frame of the development of this software, I would think that it's pretty likely that he wasn't too careful about cleaning up the code before he released it.