- User Since
- Nov 16 2017, 6:22 PM (148 w, 6 d)
May 4 2020
I'm totally cool with this change; however, it's been a while since I've taken a look at this part of MCA. I'll let other's chime in as well, but +1 from me.
Feb 21 2020
Feb 20 2020
Sep 26 2019
Sep 25 2019
Sep 18 2019
Sep 11 2019
Aug 27 2019
LGTM, nice change! I assume all of the existing LSUnit testing is sufficient.
Jun 24 2019
This looks great! Thanks.
Jun 19 2019
Awesome patch. I'm cool with this as long as Simon's comments are addressed.
Jun 11 2019
Thanks for the patch @Maxpm! Can we close PR42173? Is there anything left for that ticket?
Jun 10 2019
May 9 2019
May 8 2019
This looks nice! We should probably also have a test for the case where a user specifies an END before a BEGIN tag.
Apr 22 2019
Apr 19 2019
Thanks for the review, I addressed your comments:
- Removed the reference, and pass the AST node as a pointer.
- Made a few decls auto
Apr 17 2019
Apr 2 2019
LGTM, I'll let @jhenderson weigh-in to see what he says.
LGTM, as long as @rupprecht's comment is addressed.
Apr 1 2019
Mar 30 2019
Mar 28 2019
Mar 22 2019
Mar 13 2019
Mar 7 2019
Makes sense. LGTM.
Mar 1 2019
Thanks for the reviews everyone. This update addresses the feedback, all of which was incorporated.
Feb 28 2019
Feb 27 2019
I like this and think users will find it very helpful. The changes seem sensible to me; however, I had a few nits... mostly stylistic things, no big deals. Anyways, I'll mark this patch as accept, as long as you cover the suggestions made by @lebedev.ri.
- Fixed the test case (removed underscore typo, and extraneous shstrtab test).
- Emit ANDROID_REL ANDROID_RELA instead of section offsets.
- Modified the range check, allowing us to remove some now redundant code for printing the lower-bound section names.
Feb 26 2019
- Added a more complicated test.
- Made a few variables 'auto' instead of their iterator type.
Feb 20 2019
LGTM. Thanks for adding these.