search

albertopasqualetto / browser-paths

A simple package that retrieves the path of the most known browsers (if installed), no matter the OS
Apache License 2.0
8 stars 4 forks source link

No reason to check for existence with which.sync #1

Closed Neogavin closed 1 year ago

Neogavin commented 1 year ago

...if calling which.sync with "nothrow" set. This causes an error when not found; removing that check fixes it, and returns a null value as expected.

albertopasqualetto commented 1 year ago

Yes, you are right, thanks for your PR!