{"comment":{"url":"https://api.github.com/repos/Cyan4973/zstd/comments/17064163","html_url":"https://github.com/Cyan4973/zstd/commit/485371600a99473d68006f923ded85d9ff8be27b#commitcomment-17064163","id":17064163,"user":{"login":"Cyan4973","id":750081,"avatar_url":"https://avatars.githubusercontent.com/u/750081?v=3","gravatar_id":"","url":"https://api.github.com/users/Cyan4973","html_url":"https://github.com/Cyan4973","followers_url":"https://api.github.com/users/Cyan4973/followers","following_url":"https://api.github.com/users/Cyan4973/following{/other_user}","gists_url":"https://api.github.com/users/Cyan4973/gists{/gist_id}","starred_url":"https://api.github.com/users/Cyan4973/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Cyan4973/subscriptions","organizations_url":"https://api.github.com/users/Cyan4973/orgs","repos_url":"https://api.github.com/users/Cyan4973/repos","events_url":"https://api.github.com/users/Cyan4973/events{/privacy}","received_events_url":"https://api.github.com/users/Cyan4973/received_events","type":"User","site_admin":false},"position":83,"line":2530,"path":"lib/zstd_compress.c","commit_id":"485371600a99473d68006f923ded85d9ff8be27b","created_at":"2016-04-12T10:05:44Z","updated_at":"2016-04-12T10:05:44Z","body":"Just noticed this line.\r\nThe first test, I don't understand :  ` if (pledgedSrcSize || pledgedSrcSize >= 8192)`\r\n\r\nIf `pledgedSrcSize >= 8192`, then necessarily `pledgedSrcSize > 0`, \r\nso the first test looks redundant.\r\n"}}