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

Issue 975: Add support for removes in updater.py and update-client.py

Can't Edit
Can't Publish+Mail
Start Review
Created:
18 years ago by noe
Modified:
11 years, 5 months ago
Reviewers:
Panayiotis
Base URL:
http://malfind2.googlecode.com/svn/trunk/
Visibility:
Public.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+30 lines, -29 lines) Patch
index.yaml View 1 chunk +1 line, -1 line 0 comments Download
update-client.py View 3 chunks +14 lines, -17 lines 0 comments Download
updater.py View 2 chunks +15 lines, -11 lines 5 comments Download

Messages

Total messages: 1
Panayiotis
18 years ago (2008-05-16 22:24:48 UTC) #1
http://codereview.appspot.com/975/diff/1/4
File updater.py (right):

http://codereview.appspot.com/975/diff/1/4#newcode1
Line 1: import logging
move all these changes to display/handler_update.py :) 

(you may need to svn up)

http://codereview.appspot.com/975/diff/1/4#newcode22
Line 22: for m in re.finditer('([+-])([a-zA-Z0-9]{32})', data):
a-fA-F

also lowercase before adding / looking up

http://codereview.appspot.com/975/diff/1/4#newcode23
Line 23: op, hash = m.group(1), m.group(2)
what if this doesn't match? let's log and continue.

http://codereview.appspot.com/975/diff/1/4#newcode24
Line 24: digest = Digest.gql('WHERE hash = :1', hash).get()
name this digests, and instead of get() do get(100) (see the API I think that's
how you get all matches).

http://codereview.appspot.com/975/diff/1/4#newcode30
Line 30: del_count += 1
else logging.warning ...
Sign in to reply to this message.

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