New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Change default logcat format to threadtime #31
Conversation
@@ -13,7 +13,7 @@ | |||
"adb_snap_lines": 5, | |||
|
|||
// Max number of lines in the adb logcat buffer | |||
"adb_maxlines": 20000, | |||
"adb_maxlines": 200000, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
200000 is too much. In my experience, ADBView can barely handle such a number of lines. So I suggest to change this value to 100000.
@@ -1,7 +1,7 @@ | |||
[ | |||
{ "caption": "-", "id": "adb" }, | |||
{ "command": "adb_filter_by_process_id", "caption": "ADB Filter by Process ID" }, | |||
{ "command": "adb_filter_by_process_name", "caption": "ADB Filter by Process Name" }, | |||
{ "command": "adb_filter_by_thread_id", "caption": "ADB Filter By Thread ID" }, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The "filter by process name" feature still makes sense. Could you just add a new command for "filter by thread id", instead of replacing an existing one? Also, could you update README.md for the new added command as well?
<dict> | ||
<key>name</key> | ||
<string>Android Debug Bridge</string> | ||
<string>Android Debug Bridge Full</string> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is no need to change the name.
if match != None: | ||
set_filter(self.view, "^\d+\-\d+ [\d\:\.]* +\d+ +\d+ %s:" % match.group(1)) | ||
else: | ||
sublime.error_message("Couldn't extract thread id") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: "extract tag name" :)
Thank you! |
01-12 14:41:02.931 13186 13211 D AccountTypeManager: Registering 1 extension packages
The thread id may be useful for developers who cares about multiple-thread runtime condition.