Merge pull request #14 from bibliolabs/develop

Fix off by 1 error on CFI generation. First element is 2 not zero.
readium · Aug 2, 2013 · ff45d4c · ff45d4c
2 parents a01a58d + 838e80d
commit ff45d4c
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/ePub3/ePub/package.cpp b/ePub3/ePub/package.cpp
@@ -869,14 +869,14 @@ const CFI Package::CFIForManifestItem(shared_ptr<ManifestItem> item) const
 {
     CFI result;
     result._components.emplace_back(_spineCFIIndex);
-    result._components.emplace_back(_Str(IndexOfSpineItemWithIDRef(item->Identifier())*2, "[", item->Identifier(), "]!"));
+    result._components.emplace_back(_Str((IndexOfSpineItemWithIDRef(item->Identifier())+1)*2, "[", item->Identifier(), "]!"));
     return result;
 }
 const CFI Package::CFIForSpineItem(shared_ptr<SpineItem> item) const
 {
     CFI result;
     result._components.emplace_back(_spineCFIIndex);
-    result._components.emplace_back(_Str(item->Index()*2, "[", item->Idref(), "]!"));
+    result._components.emplace_back(_Str((item->Index()+1)*2, "[", item->Idref(), "]!"));
     return result;
 }
 shared_ptr<ManifestItem> Package::ManifestItemForCFI(ePub3::CFI &cfi, CFI* pRemainingCFI) const