Skip to content
This repository was archived by the owner on Apr 7, 2026. It is now read-only.

Fix bug - #85

Open
PhaseboyTA wants to merge 69 commits into
fix-log-parsefrom
master
Open

Fix bug#85
PhaseboyTA wants to merge 69 commits into
fix-log-parsefrom
master

Conversation

@PhaseboyTA

Copy link
Copy Markdown

What is different

edsj-sony and others added 30 commits January 23, 2020 12:29
Fix the parsing to get this to work, this needs to
be rewritten properly.
Given new log parsing features (a quite noticable change) we will bump
the VERSION_CODE to 217 so we can differentiate this from other builds.

Co-authored-by: toshi38-sony <43984100+toshi38-sony@users.noreply.github.com>
* Correctly extract the PID from the log line instead of the UID
* Update the parsing to extract the reason for a crash and
  the package name of the process that crashed
* Correctly parse the ApplicationInfo flags from crashed process
This is expected by some pre-existing tools and the new gradle build
does not include it by default.
Create main class item in manifest.
35d328aeca08abc36a6ca8ec1cc7406dab5f63a8 removed mAnimLayer so no point
in continuing to parse for it.
Procrank parsing was based on width of rows, but this doesn't hold if
there are "large" numbers that overflow to the next column.  Instead
this now parses based on whitespace.
Some graphs have quite long scales, in these cases it is better with a
day time stamp instead of hours.
Get a much longer log then we get from the event log battery events.

@PhaseboyTA PhaseboyTA left a comment

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thx

@Cokemancola69696 Cokemancola69696 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

BUG-06591c2c58d6c834e96c2b400a825de91fcbe96947a932ac1673e96ce68d74ae-20240528052937Z-f34c5cb3d8bd65e7

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants