Closed Bug 744094 Opened 8 years ago Closed 8 years ago

Robocop: Log robocop errors into the log

Categories

(Testing :: General, defect)

x86
macOS
defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED
mozilla14

People

(Reporter: kats, Assigned: kats)

References

Details

Attachments

(2 files)

I wrote up these patches as a first step to debugging bug 744049 but then jmaher figured it out. Still, no point in letting these patches go to waste...
Attachment #613692 - Flags: review?(jmaher)
Comment on attachment 613692 [details] [diff] [review]
(1/2) Shorten enum value names because the enum class already has LogLevel

Review of attachment 613692 [details] [diff] [review]:
-----------------------------------------------------------------

nice and clean, thanks.
Attachment #613692 - Flags: review?(jmaher) → review+
Comment on attachment 613693 [details] [diff] [review]
(2/2) Convert printStackTrace calls to log calls

Review of attachment 613693 [details] [diff] [review]:
-----------------------------------------------------------------

does this remove the need for bug 727354 ?
Attachment #613693 - Flags: review?(jmaher) → review+
(In reply to Joel Maher (:jmaher) from comment #4)
> does this remove the need for bug 727354 ?

Heh, yeah. I was looking for that bug before I created this one but eventually gave up.
You need to log in before you can comment on or make changes to this bug.