microsoft / thrifty

Thrift for Android that saves you methods
Apache License 2.0
543 stars 101 forks source link

Remove SecurityManager usage, use exceptions instead #532

Closed benjamin-bader closed 1 year ago

benjamin-bader commented 1 year ago

SecurityManager is deprecated and will be removed. As it happens we were using it only to intercept calls to System.exit in an integration test, while invoking the exiting method directly. We can just replace the calls to System.exit with exceptions and rip out the deprecated stuff entirely.

The code isn't pretty, but it gets the job done.