Thumbnailer

Merge lp:~jpakkane/thumbnailer/off-by-one-fix into lp:thumbnailer

  1. off-by-one-fix
  2. Merge into trunk
Proposed by Jussi Pakkanen
Status: Merged
Approved by: Jussi Pakkanen
Approved revision: 56
Merged at revision: 54
Proposed branch: lp:~jpakkane/thumbnailer/off-by-one-fix
Merge into: lp:thumbnailer
Diff against target: 12 lines (+1/-1)
1 file modified
src/thumbnailcache.cpp (+1/-1)
To merge this branch: bzr merge lp:~jpakkane/thumbnailer/off-by-one-fix
Reviewer Review Type Date Requested Status
Jamie Strandboge Approve
PS Jenkins bot (community) continuous-integration Approve
Unity Team Pending
Review via email: mp+189619@code.launchpad.net

Commit message

Fix off by one error.

Description of the change

Fix off by one error.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

PASSED: Continuous integration, rev:56
http://10.97.0.26:8080/job/thumbnailer-ci/9/
Executed test runs:
    SUCCESS: http://10.97.0.26:8080/job/thumbnailer-saucy-amd64-ci/9
    SUCCESS: http://10.97.0.26:8080/job/thumbnailer-saucy-armhf-ci/9
        deb: http://10.97.0.26:8080/job/thumbnailer-saucy-armhf-ci/9/artifact/work/output/*zip*/output.zip
    SUCCESS: http://10.97.0.26:8080/job/thumbnailer-saucy-i386-ci/9

Click here to trigger a rebuild:
http://10.97.0.26:8080/job/thumbnailer-ci/9/rebuild

review: Approve (continuous-integration)
Revision history for this message
Jamie Strandboge (jdstrand) wrote :

ACK

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/thumbnailcache.cpp'
2--- src/thumbnailcache.cpp 2013-10-02 12:02:04 +0000
3+++ src/thumbnailcache.cpp 2013-10-07 14:38:30 +0000
4@@ -109,7 +109,7 @@
5 if(ind == 0) {
6 throw runtime_error("/proc/self/attr/current malformed, starts with '_'.");
7 }
8- return core.substr(0, ind-1);
9+ return core.substr(0, ind);
10 }
11
12 class ThumbnailCachePrivate {

Subscribers

People subscribed via source and target branches

to all changes: