Thread overview
[Issue 35] Potentially wrong code with template alias parameters and nested functions
Jan 12, 2013
Johannes Pfau
Jan 12, 2013
Johannes Pfau
Jan 12, 2013
Johannes Pfau
Jan 12, 2013
Iain Buclaw
Jan 12, 2013
Johannes Pfau
Mar 01, 2013
Johannes Pfau
January 12, 2013
http://gdcproject.org/bugzilla/show_bug.cgi?id=35

--- Comment #1 from Johannes Pfau <johannespfau@gmail.com> 2013-01-12 18:17:40 UTC ---
Created attachment 25
  --> http://gdcproject.org/bugzilla/attachment.cgi?id=25
test case 2

-- 
Configure issuemail: http://gdcproject.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching all issue changes.
January 12, 2013
http://gdcproject.org/bugzilla/show_bug.cgi?id=35

--- Comment #2 from Johannes Pfau <johannespfau@gmail.com> 2013-01-12 18:18:21 UTC ---
Created attachment 26
  --> http://gdcproject.org/bugzilla/attachment.cgi?id=26
Simple patch

-- 
Configure issuemail: http://gdcproject.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching all issue changes.
January 12, 2013
http://gdcproject.org/bugzilla/show_bug.cgi?id=35

Johannes Pfau <johannespfau@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |johannespfau@gmail.com

-- 
Configure issuemail: http://gdcproject.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching all issue changes.
January 12, 2013
http://gdcproject.org/bugzilla/show_bug.cgi?id=35

--- Comment #3 from Iain Buclaw <ibuclaw@gdcproject.org> 2013-01-12 18:23:58 UTC ---
Looks fine to me, though the whitespace looks a bit off. :)

-- 
Configure issuemail: http://gdcproject.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching all issue changes.
January 12, 2013
http://gdcproject.org/bugzilla/show_bug.cgi?id=35

--- Comment #4 from Johannes Pfau <johannespfau@gmail.com> 2013-01-12 18:29:07 UTC ---
Yes, the patch was only a quick test, it's not ready for merging. It's a little bit annoying that it can't detect complicated cases as the hasNestedArgs check in the frontend doesn't detect those properly...

-- 
Configure issuemail: http://gdcproject.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching all issue changes.
March 01, 2013
http://gdcproject.org/bugzilla/show_bug.cgi?id=35

Johannes Pfau <johannespfau@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED

--- Comment #5 from Johannes Pfau <johannespfau@gmail.com> 2013-03-01 17:21:05 UTC ---
Fixed https://github.com/D-Programming-GDC/GDC/commit/111c97180ef44b3b3a3ca01afb6dd1267ba35856

-- 
Configure issuemail: http://gdcproject.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching all issue changes.