[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: T1249 - Remove (define define-ly-syntax define-public). (issue231304
From: |
pnorcks |
Subject: |
Re: T1249 - Remove (define define-ly-syntax define-public). (issue2313044) |
Date: |
Wed, 10 Nov 2010 06:01:47 +0000 |
Hi Ian,
LGTM.
Before sending me the git patch, I would recommend changing the subject
from "Remove (define define-ly-syntax define-public)" to something more
accurate that reflects what this patch fixes.
Thanks for your work on this!
Regards,
Patrick
http://codereview.appspot.com/2313044/
- Re: T1249 - Remove (define define-ly-syntax define-public). (issue2313044), ianhulin44, 2010/11/04
- Re: T1249 - Remove (define define-ly-syntax define-public). (issue2313044), pnorcks, 2010/11/04
- Re: T1249 - Remove (define define-ly-syntax define-public). (issue2313044), ianhulin44, 2010/11/04
- Re: T1249 - Remove (define define-ly-syntax define-public). (issue2313044), v . villenave, 2010/11/04
- Re: T1249 - Remove (define define-ly-syntax define-public). (issue2313044), ianhulin44, 2010/11/04
- Re: T1249 - Remove (define define-ly-syntax define-public). (issue2313044), pnorcks, 2010/11/04
- Re: T1249 - Remove (define define-ly-syntax define-public). (issue2313044), ianhulin44, 2010/11/05
- Re: T1249 - Remove (define define-ly-syntax define-public). (issue2313044),
pnorcks <=