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

Issue 9346048: code review 9346048: publicsuffix: update table to latest list from p... (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
11 years, 10 months ago by volker.dobler
Modified:
11 years, 6 months ago
Reviewers:
nigeltao
CC:
nigeltao, golang-dev
Visibility:
Public.

Description

publicsuffix: update table to latest list from publicsuffix.org Update the public suffix list to the latest (April 23. 2013) data of publicsuffix.org's list.

Patch Set 1 #

Patch Set 2 : diff -r 1e65ca1b2499 https://code.google.com/p/go.net #

Patch Set 3 : diff -r 1e65ca1b2499 https://code.google.com/p/go.net #

Unified diffs Side-by-side diffs Delta from patch set Stats (+6926 lines, -6827 lines) Patch
M publicsuffix/table.go View 1 4 chunks +6810 lines, -6772 lines 0 comments Download
M publicsuffix/table_test.go View 1 14 chunks +116 lines, -55 lines 0 comments Download

Messages

Total messages: 3
volker.dobler
Hello nigeltao@golang.org (cc: golang-dev@googlegroups.com), I'd like you to review this change to https://code.google.com/p/go.net
11 years, 10 months ago (2013-05-12 12:37:49 UTC) #1
nigeltao
LGTM.
11 years, 10 months ago (2013-05-14 04:57:19 UTC) #2
nigeltao
11 years, 10 months ago (2013-05-14 04:58:24 UTC) #3
*** Submitted as
https://code.google.com/p/go/source/detail?r=048d9bed1aa3&repo=net ***

publicsuffix: update table to latest list from publicsuffix.org

Update the public suffix list to the latest (April 23. 2013)
 data of publicsuffix.org's list.

R=nigeltao
CC=golang-dev
https://codereview.appspot.com/9346048

Committer: Nigel Tao <nigeltao@golang.org>
Sign in to reply to this message.

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