{"action":"created","issue":{"url":"https://api.github.com/repos/chrsmith/leveldb/issues/173","labels_url":"https://api.github.com/repos/chrsmith/leveldb/issues/173/labels{/name}","comments_url":"https://api.github.com/repos/chrsmith/leveldb/issues/173/comments","events_url":"https://api.github.com/repos/chrsmith/leveldb/issues/173/events","html_url":"https://github.com/chrsmith/leveldb/issues/173","id":58933767,"number":173,"title":"PosixSequentialFile::Skip(uint64_t n)","user":{"login":"GoogleCodeExporter","id":9614759,"avatar_url":"https://avatars.githubusercontent.com/u/9614759?v=3","gravatar_id":"","url":"https://api.github.com/users/GoogleCodeExporter","html_url":"https://github.com/GoogleCodeExporter","followers_url":"https://api.github.com/users/GoogleCodeExporter/followers","following_url":"https://api.github.com/users/GoogleCodeExporter/following{/other_user}","gists_url":"https://api.github.com/users/GoogleCodeExporter/gists{/gist_id}","starred_url":"https://api.github.com/users/GoogleCodeExporter/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/GoogleCodeExporter/subscriptions","organizations_url":"https://api.github.com/users/GoogleCodeExporter/orgs","repos_url":"https://api.github.com/users/GoogleCodeExporter/repos","events_url":"https://api.github.com/users/GoogleCodeExporter/events{/privacy}","received_events_url":"https://api.github.com/users/GoogleCodeExporter/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/chrsmith/leveldb/labels/auto-migrated","name":"auto-migrated","color":"ededed"},{"url":"https://api.github.com/repos/chrsmith/leveldb/labels/Priority-Medium","name":"Priority-Medium","color":"ededed"},{"url":"https://api.github.com/repos/chrsmith/leveldb/labels/Type-Enhancement","name":"Type-Enhancement","color":"ededed"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":5,"created_at":"2015-02-25T17:01:37Z","updated_at":"2015-02-25T17:01:39Z","closed_at":null,"body":"```\nThe method PosixSequentialFile::Skip(uint64_t n) uses fseek, which is defined \nas:\n\nint fseek(FILE *stream, long offset, int whence);\n\nThe problem is that \"offset\" is 4 bytes in 32bit machines.\n\nWouldn't it be better to use fseeko?\n\nint fseeko(FILE *stream, off_t offset, int whence);\n\nand in the Makefile have:\n-D_FILE_OFFSET_BITS=64\n```\n\n-----\nOriginal issue reported on code.google.com by `guido.re...@yahoo.com` on 29 May 2013 at 4:38"},"comment":{"url":"https://api.github.com/repos/chrsmith/leveldb/issues/comments/76002618","html_url":"https://github.com/chrsmith/leveldb/issues/173#issuecomment-76002618","issue_url":"https://api.github.com/repos/chrsmith/leveldb/issues/173","id":76002618,"user":{"login":"GoogleCodeExporter","id":9614759,"avatar_url":"https://avatars.githubusercontent.com/u/9614759?v=3","gravatar_id":"","url":"https://api.github.com/users/GoogleCodeExporter","html_url":"https://github.com/GoogleCodeExporter","followers_url":"https://api.github.com/users/GoogleCodeExporter/followers","following_url":"https://api.github.com/users/GoogleCodeExporter/following{/other_user}","gists_url":"https://api.github.com/users/GoogleCodeExporter/gists{/gist_id}","starred_url":"https://api.github.com/users/GoogleCodeExporter/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/GoogleCodeExporter/subscriptions","organizations_url":"https://api.github.com/users/GoogleCodeExporter/orgs","repos_url":"https://api.github.com/users/GoogleCodeExporter/repos","events_url":"https://api.github.com/users/GoogleCodeExporter/events{/privacy}","received_events_url":"https://api.github.com/users/GoogleCodeExporter/received_events","type":"User","site_admin":false},"created_at":"2015-02-25T17:01:39Z","updated_at":"2015-02-25T17:01:39Z","body":"```\nGiven how LevelDB works, it strikes me as unlikely that a 32-bit system would \nend up having files large enough for -D_FILE_OFFSET_BITS=64 or fseeko to matter.\n```\n\n-----\nOriginal comment by `cbsm...@gmail.com` on 4 Oct 2013 at 11:30"}}