{"push_id":613549509,"size":1,"distinct_size":1,"ref":"refs/heads/jit-warnings-cleanup","head":"32c2a3bc5a6b0afe26812b5bf255f43790ef979d","before":"258cc00aa9008bfd290b33c8731e8aea0c8d9d86","commits":[{"sha":"32c2a3bc5a6b0afe26812b5bf255f43790ef979d","author":{"email":"c457b3e5195ce3a91c472dc392bcc90fb508bb62@microsoft.com","name":"Jan Vorlicek"},"message":"Fix warnings in the jitter code\n\nThis change fixes some of the warnings in the jitter code on Linux. With this change\ncombined with an upcoming change in the rest of the codebase, 16 warning disabling\noptions can be removed.\n\nThe issues in the jitter were:\n1) Incorrect typedefs for const_reference and const_pointer (the const was ignored)\n2) Member initiazalization order in class constructor didn't correspond to the member order in some classes\n3) Objects with vtables were copied via memcpy. While this is intentional, cast of the pointers to void*\n   is required to silence clang warning\n4) Comparing values of different enums - there are cases of two enums containing the same values.\n5) Casting int to pointer - the cast was legal (forming a handle), adding cast to size_t in between used to fix the warning\n6) \"static struct\" - removed the static keyword\n7) Missing return value in methods with _ASSERTE\n8) Class name classifier on methods in the .h\n9) Specialized template member functions need to be defined out of the class","distinct":true,"url":"https://api.github.com/repos/janvorli/coreclr/commits/32c2a3bc5a6b0afe26812b5bf255f43790ef979d"}]}