[m-rev.] trivial diff: document a problem with the unused imports pass

Julien Fischer juliensf at csse.unimelb.edu.au
Tue Dec 19 15:58:41 AEDT 2006


Estimated hours taken: 0
Branches: main

compiler/unused_imports.m:
 	Document a problem with the analysis carried out by this module.

Julien.

Index: unused_imports.m
===================================================================
RCS file: /home/mercury/mercury1/repository/mercury/compiler/unused_imports.m,v
retrieving revision 1.6
diff -u -r1.6 unused_imports.m
--- unused_imports.m	1 Dec 2006 15:04:28 -0000	1.6
+++ unused_imports.m	19 Dec 2006 04:56:59 -0000
@@ -16,6 +16,12 @@
  % It also determines which imports aren't used in the interface, and thus
  % should only be in the implementation.
  %
+% XXX The analysis carried out here can break in the presence of procedures
+% that have both foreign language clauses and default Mercury clauses.
+% The problem is that some of module imports may only be required by
+% the Mercury clauses but these are thrown away if we use the foreign clauses.
+% As a result the module imports may appear to be unused.
+%
  %-----------------------------------------------------------------------------%

  :- module check_hlds.unused_imports.

--------------------------------------------------------------------------
mercury-reviews mailing list
Post messages to:       mercury-reviews at csse.unimelb.edu.au
Administrative Queries: owner-mercury-reviews at csse.unimelb.edu.au
Subscriptions:          mercury-reviews-request at csse.unimelb.edu.au
--------------------------------------------------------------------------



More information about the reviews mailing list