)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"change_message_id":"22df958ae032260a6865cf440df6dbeb2b9d31be","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"3506c06d_f3989c8f","updated":"2026-06-16 17:32:07.000000000","message":"i like the variable name over the comment - I think this is an improvement.\n\nsaying *something* about `app_iter_ranges` wouldn\u0027t have been too bad, but hopefully blame will point us back to the story if we ever need it.","commit_id":"bab1688e2c91cb9866c714ee77bb8c45f6885e7b"}],"swift/obj/diskfile.py":[{"author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"change_message_id":"22df958ae032260a6865cf440df6dbeb2b9d31be","unresolved":true,"context_lines":[{"line_number":2338,"context_line":"        else:"},{"line_number":2339,"context_line":"            length \u003d None"},{"line_number":2340,"context_line":"        try:"},{"line_number":2341,"context_line":"            for chunk in self:"},{"line_number":2342,"context_line":"                if length is not None:"},{"line_number":2343,"context_line":"                    length -\u003d len(chunk)"},{"line_number":2344,"context_line":"                    if length \u003c 0:"}],"source_content_type":"text/x-python","patch_set":2,"id":"aa3c85dd_55a244ce","line":2341,"updated":"2026-06-16 17:32:07.000000000","message":"the tricky bit is really that `app_iter_range` calls `seek` and then proxies into the un-parameterized `__iter__` \n\nso there\u0027s no good way to \"pass\" the value of the start \"into\" `_inner_iter` - better to just use the `self._fp` \"state\" (I don\u0027t *think* that python\u0027s `tell` method is *actually* doing an `lseek(fd, 0, SEEK_CUR)` syscall to lookup the current file pos but I could be wrong)","commit_id":"bab1688e2c91cb9866c714ee77bb8c45f6885e7b"}]}