[m-rev.] Refactoring make_temp
Sebastian Godelet
sebastian.godelet at outlook.com
Thu May 26 16:28:40 AEST 2016
Hi Paul,
>From an initial screening I think you preserved all the additions I made, but I've yet to build this.
Couple of things though:
In line 10935 (https://github.com/Mercury-Language/mercury/compare/master...PaulBone:make_temp#diff-c8358543868f79aa0b536505b8138fbaR10935) you comment out a huge bunch of code,
I'd rather rely on version management instead (so git blame and stuff continue to work); also I think io.m is already large enough.
In line 11226, you changed have_make_temp_directory to always return MR_TRUE, maybe we should get rid off the procedure altogether (if I remember that was a new, so unreleased, addition).
I'll try to build this changes later
Sebastian.
P.S: I hope I didn't send emails twice or more times my Outlook was giving up on me.
More information about the reviews
mailing list