{"push_id":874697459,"size":5,"distinct_size":5,"ref":"refs/heads/waitAndGetResult","head":"19a813c581f1c0584ee1661ef2de9b3c276a713d","before":"24bf972460a0312f260b337bf6b47e3e712c5c8e","commits":[{"sha":"93902d9fa19d152542d97ca0a60ad70b3760d510","author":{"email":"a153fe40cf23404581b29e8537095154d0b71ac0@microsoft.com","name":"Jared Parsons"},"message":"Clarify overflow checks\n\nThe intent of the code base was to be run with overflow checking disabled.  However the actual settings were quite a bit different:\n\n- CSharp: checking enabled in debug, disabled in release\n- VB: checking disabled in all modes\n\nFor both languages individual projects could, and often did, override this setting.  Typically it specified the global default but not always.\n\nThis is confusing and makes it difficult to reason about our source code.  Furthermore much of our testing occurs in debug mode and it's not logical for our code to behave substantially different in Debug than it would in Release.\n\nThis change clarifies our overflow checking behavior.  It's disabled in all modes now.  The change goes further to prevent individual projects from overriding this setting and adds unit tests to make sure it was applied correctly.\n\nNote: this PR is not intended to state whether it's better for overflow checking to be enabled or disabled.  It is intended to enforce the decision the code base was designed around.","distinct":true,"url":"https://api.github.com/repos/CyrusNajmabadi/roslyn/commits/93902d9fa19d152542d97ca0a60ad70b3760d510"},{"sha":"f9d2867dc0cd5c45072f596e1f69d7cce71dd963","author":{"email":"32ad247f77b8a066ef05467ce49a5a63e193c3a3@pre-alpha.com","name":"Ty Overby"},"message":"Don't copy a massive byte-array for fuzz testing","distinct":true,"url":"https://api.github.com/repos/CyrusNajmabadi/roslyn/commits/f9d2867dc0cd5c45072f596e1f69d7cce71dd963"},{"sha":"9bb8aab9d0566426a62928da9ee9fb98b3325f29","author":{"email":"d131939c23330f3c945ed5437217aca3c84079c2@gmail.com","name":"Jared Parsons"},"message":"Merge pull request #6933 from jaredpar/overflow\n\nClarify overflow checks","distinct":true,"url":"https://api.github.com/repos/CyrusNajmabadi/roslyn/commits/9bb8aab9d0566426a62928da9ee9fb98b3325f29"},{"sha":"abaeff6adb2b1e68df71e21bee49d77fae134c60","author":{"email":"d131939c23330f3c945ed5437217aca3c84079c2@gmail.com","name":"Jared Parsons"},"message":"Merge pull request #6916 from TyOverby/6744-no-copy-fuzz\n\nDon't copy a massive byte-array for fuzz testing","distinct":true,"url":"https://api.github.com/repos/CyrusNajmabadi/roslyn/commits/abaeff6adb2b1e68df71e21bee49d77fae134c60"},{"sha":"19a813c581f1c0584ee1661ef2de9b3c276a713d","author":{"email":"d730d825a7ed112a6b950b3a94d1011b757d272e@microsoft.com","name":"Cyrus Najmabadi"},"message":"Merge remote-tracking branch 'upstream/master' into waitAndGetResult","distinct":true,"url":"https://api.github.com/repos/CyrusNajmabadi/roslyn/commits/19a813c581f1c0584ee1661ef2de9b3c276a713d"}]}