Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(1282)

Issue 278260043: Multiple fixes for Windows Registry parsing #416 (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
8 years, 4 months ago by Joachim Metz
Modified:
8 years, 4 months ago
Reviewers:
Joachim Metz, kiddi, onager
CC:
log2timeline-dev_googlegroups.com
Visibility:
Public.

Description

Multiple fixes for Windows Registry parsing #416

Patch Set 1 #

Total comments: 1

Patch Set 2 : Multiple fixes for Windows Registry parsing #416 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+52 lines, -21 lines) Patch
M plaso/parsers/winreg.py View 1 3 chunks +41 lines, -15 lines 0 comments Download
M plaso/parsers/winreg_plugins/appcompatcache.py View 1 chunk +2 lines, -2 lines 0 comments Download
M plaso/parsers/winreg_plugins/mrulistex.py View 2 chunks +9 lines, -4 lines 0 comments Download

Messages

Total messages: 4
Joachim Metz
8 years, 4 months ago (2015-11-28 17:03:45 UTC) #1
Joachim Metz
https://codereview.appspot.com/278260043/diff/1/plaso/parsers/winreg.py File plaso/parsers/winreg.py (right): https://codereview.appspot.com/278260043/diff/1/plaso/parsers/winreg.py#newcode90 plaso/parsers/winreg.py:90: """Parses the Registry keys recursively. fix docstring.
8 years, 4 months ago (2015-11-28 17:04:55 UTC) #2
Joachim Metz
Code updated.
8 years, 4 months ago (2015-11-28 17:13:58 UTC) #3
onager
8 years, 4 months ago (2015-11-29 21:40:00 UTC) #4
On 2015/11/28 17:13:58, Joachim Metz wrote:
> Code updated.

LGTM
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b