Thompson, David <[email protected]> writes: > On Tue, Jul 21, 2015 at 9:57 AM, Ricardo Wurmus > <[email protected]> wrote: >> Something’s wrong with the alignment of the ‘(replace 'install ...)’. >> If there are no objections I can fix the alignment and push without >> requesting another version of the patch. >> >> But I think I better wait for other comments. > > Looks fine to me, sans the alignment. One additional nitpick is the > 'string-append' calls. The newlines should happen *after* the first > argument, not before.
I fixed the alignment and made a couple of small modifications to prettify the ‘string-append’ calls. Pushed as 8dc797f. Thanks again! ~~ Ricardo
