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

Issue 5375098: Add API "focusCandidate(size_t i)". (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
12 years, 6 months ago by hsumita
Modified:
12 years, 5 months ago
Reviewers:
shawn.p.huang, penghuang
CC:
komatsu, mukai, yusukes, nona, horo
Base URL:
git@github.com:pyzy/pyzy.git@master
Visibility:
Public.

Description

Add API "focusCandidate(size_t i)". BUG= None TEST= Build a package.

Patch Set 1 #

Patch Set 2 : Modify print format from %ud to %zd for size_t. #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -1 line) Patch
M src/PyZyInputContext.h View 1 chunk +1 line, -0 lines 1 comment Download
M src/PyZyPhoneticContext.cc View 1 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 4
hsumita
12 years, 6 months ago (2011-11-16 03:38:01 UTC) #1
Peng
http://codereview.appspot.com/5375098/diff/3/src/PyZyInputContext.h File src/PyZyInputContext.h (right): http://codereview.appspot.com/5375098/diff/3/src/PyZyInputContext.h#newcode82 src/PyZyInputContext.h:82: virtual bool focusCandidate (size_t index) = 0; Which class ...
12 years, 6 months ago (2011-11-17 01:38:11 UTC) #2
hsumita
I am sorry that my explanation was not enough. This function is already implemented in ...
12 years, 6 months ago (2011-11-17 03:19:28 UTC) #3
Peng
12 years, 6 months ago (2011-11-17 04:03:22 UTC) #4
lgtm
Sign in to reply to this message.

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