2013-06-24, 05:06
Ignore my comment about the regex thingy. It works just fine as is. (Aside from not using a global string constant for the regex that is used in multiple places.
I did notice that 1.4's Bulk Renamer dialog is broken atm. The underlying cause is that Ember generated .NFO files has loads of leading and trailing whitespace for the audio/video fields like aspectratio, etc...
My thought at the moment was to just always Trim in clsAPINFO.vb:GetBestVideo and GetBestAudio as well as clsAPINumUtils.cs:ConvertToSingle.
Sound good or should I do something else before submitting a pull request for it?
Thanks,
Bill
I did notice that 1.4's Bulk Renamer dialog is broken atm. The underlying cause is that Ember generated .NFO files has loads of leading and trailing whitespace for the audio/video fields like aspectratio, etc...
My thought at the moment was to just always Trim in clsAPINFO.vb:GetBestVideo and GetBestAudio as well as clsAPINumUtils.cs:ConvertToSingle.
Sound good or should I do something else before submitting a pull request for it?
Thanks,
Bill