jockeyjs icon indicating copy to clipboard operation
jockeyjs copied to clipboard

replace usage of e.printStackTrace() with Log.e(TAG, MESSAGE, e)

Open tomerlevi444 opened this issue 10 years ago • 1 comments

JockeyJS is mostly hosted as an external tool inside apps, and therefore, printing stacktrace is not recommended. Instead, you should tag the exception (to signal that it originated from JockeyJS's library). This is also recommended in the official Android documentation: http://developer.android.com/reference/android/util/Log.html

BTW, Log.e prints the stacktrace for you.

tomerlevi444 avatar Jan 25 '15 14:01 tomerlevi444

Thanks @tomerlevi444. I'll take a look, I think I might have just been lazy using eclipse's snippets.

paulpdaniels avatar Feb 09 '15 21:02 paulpdaniels