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

Issue 4523057: venti: import changes from plan 9

Can't Edit
Can't Publish+Mail
Start Review
Created:
13 years, 8 months ago by 0intro
Modified:
13 years, 7 months ago
Reviewers:
CC:
rsc, codebot
Visibility:
Public.

Patch Set 1 #

Total comments: 1

Patch Set 2 : venti: import changes from plan 9 #

Patch Set 3 : venti: import changes from plan 9 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+142 lines, -64 lines) Patch
M src/cmd/venti/dump.c View 1 chunk +0 lines, -1 line 0 comments Download
M src/cmd/venti/root.c View 3 chunks +8 lines, -2 lines 0 comments Download
M src/cmd/venti/srv/arena.c View 1 chunk +4 lines, -1 line 0 comments Download
M src/cmd/venti/srv/arenas.c View 1 2 3 chunks +7 lines, -2 lines 0 comments Download
M src/cmd/venti/srv/buildindex.c View 4 chunks +17 lines, -4 lines 0 comments Download
M src/cmd/venti/srv/checkarenas.c View 1 chunk +3 lines, -1 line 0 comments Download
M src/cmd/venti/srv/config.c View 2 chunks +2 lines, -2 lines 0 comments Download
M src/cmd/venti/srv/conv.c View 3 chunks +8 lines, -2 lines 0 comments Download
M src/cmd/venti/srv/dat.h View 1 2 chunks +2 lines, -0 lines 0 comments Download
M src/cmd/venti/srv/findscore.c View 1 chunk +1 line, -1 line 0 comments Download
M src/cmd/venti/srv/fmtarenas.c View 1 chunk +1 line, -1 line 0 comments Download
M src/cmd/venti/srv/fmtindex.c View 1 chunk +2 lines, -2 lines 0 comments Download
M src/cmd/venti/srv/icachewrite.c View 1 chunk +0 lines, -1 line 0 comments Download
M src/cmd/venti/srv/index.c View 1 chunk +10 lines, -6 lines 0 comments Download
M src/cmd/venti/srv/printarena.c View 2 chunks +2 lines, -2 lines 0 comments Download
M src/cmd/venti/srv/printarenapart.c View 1 chunk +1 line, -1 line 0 comments Download
M src/cmd/venti/srv/rdarena.c View 4 chunks +9 lines, -4 lines 0 comments Download
M src/cmd/venti/srv/syncindex.c View 1 chunk +1 line, -1 line 0 comments Download
M src/cmd/venti/srv/venti.c View 3 chunks +13 lines, -1 line 0 comments Download
M src/cmd/venti/srv/wrarena.c View 3 chunks +4 lines, -4 lines 0 comments Download
M src/libavl/avl.c View 1 2 6 chunks +47 lines, -25 lines 0 comments Download

Messages

Total messages: 8
0intro
13 years, 8 months ago (2011-05-11 21:21:57 UTC) #1
rsc
http://codereview.appspot.com/4523057/diff/1/src/cmd/venti/srv/dat.h File src/cmd/venti/srv/dat.h (right): http://codereview.appspot.com/4523057/diff/1/src/cmd/venti/srv/dat.h#newcode57 src/cmd/venti/srv/dat.h:57: Unspecified = ~0ul, This is wrong. Since the memory ...
13 years, 8 months ago (2011-05-12 03:46:43 UTC) #2
0intro
13 years, 8 months ago (2011-05-12 17:10:20 UTC) #3
0intro
13 years, 8 months ago (2011-05-12 17:18:08 UTC) #4
0intro
Thank you for the fix, I pushed it back to Plan 9.
13 years, 8 months ago (2011-05-13 21:25:56 UTC) #5
0intro
13 years, 7 months ago (2011-06-01 22:38:52 UTC) #6
0intro
I fixed a couple of oversights and imported recent changes in Plan 9's libavl.
13 years, 7 months ago (2011-06-01 22:44:44 UTC) #7
rsc
13 years, 7 months ago (2011-06-02 13:34:03 UTC) #8
*** Submitted as c51118493ec5 ***

venti: import changes from plan 9

R=rsc
CC=plan9port.codebot
http://codereview.appspot.com/4523057

Committer: Russ Cox <rsc@swtch.com>
Sign in to reply to this message.

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