)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":38496,"name":"Andressa Cabistani","display_name":"Andressa","email":"acabistani@gmail.com","username":"andressadotpy","status":"I\u0027m a Software Engineer at Red Hat and I love Open Source and connect with people! Feel free to DM through IRC, I\u0027ll be delighted to chat"},"change_message_id":"8cbe1360a3fcbd391e27d3fd80ea1c39647105dc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"8855f77b_96c3e8ed","updated":"2026-07-21 09:55:20.000000000","message":"Looks good to me!","commit_id":"532fe0dd950c6dae48cb9f6594c97531413575a3"}],"swift/common/wsgi_gunicorn.py":[{"author":{"_account_id":38496,"name":"Andressa Cabistani","display_name":"Andressa","email":"acabistani@gmail.com","username":"andressadotpy","status":"I\u0027m a Software Engineer at Red Hat and I love Open Source and connect with people! Feel free to DM through IRC, I\u0027ll be delighted to chat"},"change_message_id":"8cbe1360a3fcbd391e27d3fd80ea1c39647105dc","unresolved":true,"context_lines":[{"line_number":754,"context_line":"        m_trim_threshold, m_mmap_threshold \u003d -1, -3"},{"line_number":755,"context_line":"        return bool(libc.mallopt(m_trim_threshold, 128 * 1024 * 1024) and"},{"line_number":756,"context_line":"                    libc.mallopt(m_mmap_threshold, 4 * 1024 * 1024))"},{"line_number":757,"context_line":"    except (OSError, AttributeError, TypeError):"},{"line_number":758,"context_line":"        return False"},{"line_number":759,"context_line":""},{"line_number":760,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"a84f83f4_9d0cf6d3","line":757,"updated":"2026-07-21 09:55:20.000000000","message":"Suggestion: The `import ctypes` is inside the try block, but ImportError is not caught. If ctypes is unavailable, this would propagate up and crash gunicorn startup rather than gracefully skipping the optimization. In practice this is extremely unlikely on Swift\u0027s target platform, but it would be consistent with the existing defensive approach for non-glibc platforms. So you could add ImportError to the exceptions, but I\u0027ll defer it to you because I don\u0027t feel too strong about it.","commit_id":"532fe0dd950c6dae48cb9f6594c97531413575a3"}]}
