On Fri, Nov 1, 2013 at 6:47 PM, Dmitri Gribenko <[email protected]> wrote:
> On Thu, Oct 31, 2013 at 6:47 PM, Richard Smith <[email protected]> wrote:
>> I had a go at writing a test for this (attached). However... my test fails
>> both with and without your change.
>
> Indeed. Seems like it fails because:
> - recursive_visibility_a.inner is not imported (only recursive_visibility_a
> is),
> - thus the 'inner' submodule is not showing up in any of the import lists,
> - and because of this getExportedModules() is not returning the
> correct module set -- it only considers modules that are imported.
Oh, and this means that this modification to your testcase:
-@import recursive_visibility_a;
+@import recursive_visibility_a.inner;
Actually produces a testcase that used to fail, but now passes.
Dmitri
--
main(i,j){for(i=2;;i++){for(j=2;j<i;j++){if(!(i%j)){j=0;break;}}if
(j){printf("%d\n",i);}}} /*Dmitri Gribenko <[email protected]>*/
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
