February 12, 2012 [dmd-internals] [D-Programming-Language/dmd] e36d87: fully recreate old selective import behavior... | ||||
---|---|---|---|---|
| ||||
Branch: refs/heads/master Home: https://github.com/D-Programming-Language/dmd Commit: e36d878f3aa5bf9ee93a9c8df458a3b992fde803 https://github.com/D-Programming-Language/dmd/commit/e36d878f3aa5bf9ee93a9c8df458a3b992fde803 Author: dawg <dawg at dawgfoto.de> Date: 2012-02-12 (Sun, 12 Feb 2012) Changed paths: M src/import.c A test/compilable/imports/test72a.d A test/compilable/imports/test72b.d A test/compilable/imports/test72c.d A test/compilable/test72.d Log Message: ----------- fully recreate old selective import behavior... ...which is known to be awfully broken but sets us back to 2.057. - Explicitly set the protection of the selective aliases to public. This is needed because we formerly didn't had alias protection. But now selective imports with explicit private protection fail access checks in ScopeDsymbol::search before merging them into overload sets. Commit: f330b950171adb67d2b648885c528b2fd5909130 https://github.com/D-Programming-Language/dmd/commit/f330b950171adb67d2b648885c528b2fd5909130 Author: Walter Bright <walter at walterbright.com> Date: 2012-02-12 (Sun, 12 Feb 2012) Changed paths: M src/import.c A test/compilable/imports/test72a.d A test/compilable/imports/test72b.d A test/compilable/imports/test72c.d A test/compilable/test72.d Log Message: ----------- Merge pull request #713 from dawgfoto/FixImportChanges fully recreate old selective import behavior... Compare: https://github.com/D-Programming-Language/dmd/compare/c61ab2a...f330b95 |
February 13, 2012 [dmd-internals] [D-Programming-Language/dmd] e36d87: fully recreate old selective import behavior... | ||||
---|---|---|---|---|
| ||||
Posted in reply to GitHub | Does this mean that the fix for bug 314 have been rolled back? Should bug 314 be reopen? :S http://d.puremagic.com/issues/show_bug.cgi?id=314 GitHub, el 12 de febrero a las 23:45 me escribiste: > Branch: refs/heads/master > Home: https://github.com/D-Programming-Language/dmd > Commit: e36d878f3aa5bf9ee93a9c8df458a3b992fde803 > https://github.com/D-Programming-Language/dmd/commit/e36d878f3aa5bf9ee93a9c8df458a3b992fde803 > Author: dawg <dawg at dawgfoto.de> > Date: 2012-02-12 (Sun, 12 Feb 2012) > > Changed paths: > M src/import.c > A test/compilable/imports/test72a.d > A test/compilable/imports/test72b.d > A test/compilable/imports/test72c.d > A test/compilable/test72.d > > Log Message: > ----------- > fully recreate old selective import behavior... > > ...which is known to be awfully broken but sets us back to 2.057. > > - Explicitly set the protection of the selective aliases to public. > This is needed because we formerly didn't had alias protection. > But now selective imports with explicit private protection fail > access checks in ScopeDsymbol::search before merging them into > overload sets. > > > Commit: f330b950171adb67d2b648885c528b2fd5909130 > https://github.com/D-Programming-Language/dmd/commit/f330b950171adb67d2b648885c528b2fd5909130 > Author: Walter Bright <walter at walterbright.com> > Date: 2012-02-12 (Sun, 12 Feb 2012) > > Changed paths: > M src/import.c > A test/compilable/imports/test72a.d > A test/compilable/imports/test72b.d > A test/compilable/imports/test72c.d > A test/compilable/test72.d > > Log Message: > ----------- > Merge pull request #713 from dawgfoto/FixImportChanges > > fully recreate old selective import behavior... > > > Compare: https://github.com/D-Programming-Language/dmd/compare/c61ab2a...f330b95 > _______________________________________________ > dmd-internals mailing list > dmd-internals at puremagic.com > http://lists.puremagic.com/mailman/listinfo/dmd-internals -- Leandro Lucarella (AKA luca) http://llucax.com.ar/ ---------------------------------------------------------------------- GPG Key: 5F5A8D05 (F8CD F9A7 BF00 5431 4145 104C 949E BFB6 5F5A 8D05) ---------------------------------------------------------------------- CARANCHO OBNUBILADO APARECE EN PARQUE CHACABUCO! -- Cr?nica TV |
Copyright © 1999-2021 by the D Language Foundation