From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail4-relais-sop.national.inria.fr (mail4-relais-sop.national.inria.fr [192.134.164.105]) by walapai.inria.fr (8.13.6/8.13.6) with ESMTP id p8GGRq7O013722 for ; Fri, 16 Sep 2011 18:27:52 +0200 X-IronPort-Anti-Spam-Filtered: true X-IronPort-Anti-Spam-Result: Ap8EAAx4c05beTnJ/2dsb2JhbABBhFWkAIFTAQEFIw8BRQEQCQIYAgIFFgsCAgkDAgECAUUGDQEHAod1ommRb4EshDuBEQSTSooThxs X-IronPort-AV: E=Sophos;i="4.68,394,1312149600"; d="scan'208";a="109369604" Received: from krypton.hackadomia.org (HELO dogguy.org) ([91.121.57.201]) by mail4-smtp-sop.national.inria.fr with ESMTP/TLS/ADH-AES256-SHA; 16 Sep 2011 18:27:47 +0200 X-Spam-Checker-Version: SpamAssassin 3.2.5 (2008-06-10) on krypton X-Spam-Level: Message-ID: <4E737745.2070205@dogguy.org> Date: Fri, 16 Sep 2011 18:20:21 +0200 From: Mehdi Dogguy User-Agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.1.16) Gecko/20110818 Icedove/3.0.11 MIME-Version: 1.0 To: Jun Furuse CC: caml-list References: In-Reply-To: X-Enigmail-Version: 1.0.1 OpenPGP: id=8C2ED8FF Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Subject: Re: [Caml-list] A small patch for ocamlc 3.12.1 to find reudundant opens On 09/16/2011 04:59 PM, Jun Furuse wrote: > Hi, > > I have written a patch to detect redundant opens in OCaml source > code, which is useful to keep your opened module list minimum. > Keeping the list smaller is good to avoid name space contamination. > It looks interesting. Thanks for your contribution ! Can you open a bugreport on mantis [1] and attach the patch there? so that it doesn't get lost. [1] http://caml.inria.fr/mantis/ > > […] > > I have found nearly 150 redundant opens in OCaml source code! You > should check your OCaml code with it, too! > Interesting :) Kind regards, P.S.: fwiw, I didn't try it (yet). -- Mehdi Dogguy مهدي الدڤي http://dogguy.org/