doy / spreadsheet-parsexlsx

parse XLSX files
http://metacpan.org/release/Spreadsheet-ParseXLSX
27 stars 35 forks source link

feature request: reduce crypto dependency #80

Open ft-lie opened 6 years ago

ft-lie commented 6 years ago

changing use Spreadsheet::ParseXLSX::Decryptor; to require Spreadsheet::ParseXLSX::Decryptor;

and locating it inside the _check_signature -test (row 37), will reduce direct dependency of crypto, and simplify export of bundles including this module.

There are of course pros and cons of doing this.