Rockbox

This is the bug/patch tracker for Rockbox. Click here for more information.

Quick links: Bugs · Patches · Rockbox frontpage

Tasklist

FS#10361 - Error log for rbutil for better error reports.

Attached to Project: Rockbox
Opened by Dominik Wenger (Domonoky) - Sunday, 21 June 2009, 12:09 GMT+2
Last edited by Dominik Wenger (Domonoky) - Saturday, 15 August 2009, 19:02 GMT+2
Task Type Patches
Category Rbutil
Status Closed
Assigned To No-one
Player Type All players
Severity Low
Priority Normal
Reported Version Version 3.3
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Private No

Details

This patch should help rbutil users to make better error reports.

If a Error happens in the Progresslogger, it offers a "save Log" Button, which saves the Progresslogger log, the system info and the system trace.

This patch uses the system trace patch from bluebrother.

   rbutil-errorlog.diff (28.3 KiB)
 rbutil/rbutilqt/sysinfo.h             |    3 
 rbutil/rbutilqt/rbutilqtfrm.ui        |  139 ++++++++++++++++++++--------------
 rbutil/rbutilqt/progressloggerfrm.ui  |   67 +++++++++-------
 rbutil/rbutilqt/rbutilqt.cpp          |   12 ++
 rbutil/rbutilqt/progressloggergui.h   |    1 
 rbutil/rbutilqt/main.cpp              |    4 
 rbutil/rbutilqt/rbutilqt.pro          |    8 +
 rbutil/rbutilqt/progressloggergui.cpp |   52 ++++++++++++
 rbutil/rbutilqt/rbutilqt.h            |    3 
 rbutil/rbutilqt/sysinfo.cpp           |   11 +-
 10 files changed, 208 insertions(+), 92 deletions(-)

This task depends upon

Closed by  Dominik Wenger (Domonoky)
Saturday, 15 August 2009, 19:02 GMT+2
Reason for closing:  Accepted
Additional comments about closing:  commited with some small modifications.
Comment by Dominik Riebeling (bluebrother) - Sunday, 21 June 2009, 15:12 GMT+2
The patch does not add systrace.{cpp,h} and systracefrm.ui. Are those the same files as in my systrace patch (at http://www.alice-dsl.net/dominik.riebeling/rockbox/rbutil-systrace-1.diff, isn't on the tracker yet)? Does it build up on that patch?
Comment by Dominik Riebeling (bluebrother) - Sunday, 21 June 2009, 15:13 GMT+2
Ok, missed the last sentence before posting ... :o
Comment by Dominik Wenger (Domonoky) - Sunday, 21 June 2009, 15:20 GMT+2
Ups i missed this files.
Attached patch should also contain this.

And yes it builds up upon your patch. (i did some small modifications to it).
   rbutil-errorlog.diff (32.2 KiB)
 rbutil/rbutilqt/sysinfo.h             |    3 
 rbutil/rbutilqt/rbutilqtfrm.ui        |  139 ++++++++++++++++++++--------------
 rbutil/rbutilqt/progressloggerfrm.ui  |   67 +++++++++-------
 rbutil/rbutilqt/rbutilqt.cpp          |   12 ++
 rbutil/rbutilqt/progressloggergui.h   |    1 
 rbutil/rbutilqt/main.cpp              |    4 
 rbutil/rbutilqt/rbutilqt.pro          |    8 +
 rbutil/rbutilqt/systrace.h            |   47 +++++++++++
 rbutil/rbutilqt/progressloggergui.cpp |   52 ++++++++++++
 rbutil/rbutilqt/rbutilqt.h            |    3 
 rbutil/rbutilqt/sysinfo.cpp           |   11 +-
 rbutil/rbutilqt/systrace.cpp          |   65 +++++++++++++++
 12 files changed, 320 insertions(+), 92 deletions(-)

Loading...