From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 3881 invoked by alias); 29 Sep 2014 13:02:50 -0000 Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org Received: (qmail 3715 invoked by uid 89); 29 Sep 2014 13:02:49 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-3.2 required=5.0 tests=AWL,BAYES_00,RCVD_IN_DNSWL_LOW,RP_MATCHES_RCVD,SPF_PASS autolearn=unavailable version=3.3.2 X-HELO: mail-oi0-f51.google.com Received: from mail-oi0-f51.google.com (HELO mail-oi0-f51.google.com) (209.85.218.51) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES128-SHA encrypted) ESMTPS; Mon, 29 Sep 2014 13:02:44 +0000 Received: by mail-oi0-f51.google.com with SMTP id e131so2883260oig.24 for ; Mon, 29 Sep 2014 06:02:41 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:mime-version:date:message-id:subject:from:to:cc :content-type; bh=41g6ZZKA6dN9jh5/XY+xLeGYfc8zaBRbl+1/UVgmigo=; b=ZBcbFqZc6PLYh8e2+yeMjCHOAPWYcRXRZ4U2uolE8MnOma4ipyQPWLqxo1vZVXyL5n zbg4o5xTeurLUyOQxYZyc68PpgbcO67UDi5V3hmTiD28lJ8qVMva1sDbpwZ4w69k6zDK OzCjJu7ruH+//QjGW0Onjs/tFbuRJXfNmankVKNlYIo4mHk0K19rx2IzIFav/wplL7RG QW93iE8uSMdYSPubBCgggspNRQA9whTeXl8ADGsvSb1HsLVjXJZlm4Ve9/22kNnTKMRa hO/6xvkxk0ZyBKd6RVbTlkbiCJqXvEJr/dspMmy1VRs91PuFjiGoFxCE+pjoMITFPIBA ZNGw== X-Gm-Message-State: ALoCoQkKa7P4IZUjw72oKmUiHcZxkmF2uWhZd/hD11x1IhYfRI5q1Eu2+qiyAwfsmzZS2YgziZ+I MIME-Version: 1.0 X-Received: by 10.182.20.242 with SMTP id q18mr39792903obe.52.1411995761748; Mon, 29 Sep 2014 06:02:41 -0700 (PDT) Received: by 10.202.225.135 with HTTP; Mon, 29 Sep 2014 06:02:40 -0700 (PDT) Date: Mon, 29 Sep 2014 13:02:00 -0000 Message-ID: Subject: [libstdc++] Refactor python/hook.in From: Siva Chandra To: gcc-patches , "libstdc++" Cc: Jonathan Wakely , Tom Tromey Content-Type: multipart/mixed; boundary=e89a8f83a883cdecdc050433e156 X-IsSubscribed: yes X-SW-Source: 2014-09/txt/msg02518.txt.bz2 --e89a8f83a883cdecdc050433e156 Content-Type: text/plain; charset=UTF-8 Content-length: 562 The attached patch refactors python/hook.in so that there are no individual function calls to load pretty printers and xmethods. This was suggested by Tom here: https://gcc.gnu.org/ml/gcc-patches/2014-08/msg02589.html. He indicates that it is better to put as little as possible in the hook file. The attached patch removes all code which explicitly loads the hooks from hook.in. 2014-09-29 Siva Chandra Reddy * python/hook.in: Only import libstdcxx.v6. * python/libstdcxx/v6/__init__.py: Load printers and xmethods. --e89a8f83a883cdecdc050433e156 Content-Type: text/plain; charset=US-ASCII; name="refactor_loading_hooks_v1.txt" Content-Disposition: attachment; filename="refactor_loading_hooks_v1.txt" Content-Transfer-Encoding: base64 X-Attachment-Id: f_i0ntpfnx0 Content-length: 2981 ZGlmZiAtLWdpdCBhL2xpYnN0ZGMrKy12My9weXRob24vaG9vay5pbiBiL2xp YnN0ZGMrKy12My9weXRob24vaG9vay5pbgppbmRleCBhZWIxY2RiLi4zMGNm NTM4IDEwMDY0NAotLS0gYS9saWJzdGRjKystdjMvcHl0aG9uL2hvb2suaW4K KysrIGIvbGlic3RkYysrLXYzL3B5dGhvbi9ob29rLmluCkBAIC01NSwxOCAr NTUsNCBAQCBpZiBnZGIuY3VycmVudF9vYmpmaWxlICgpIGlzIG5vdCBOb25l OgogICAgIGlmIG5vdCBkaXJfIGluIHN5cy5wYXRoOgogICAgICAgICBzeXMu cGF0aC5pbnNlcnQoMCwgZGlyXykKIAotIyBMb2FkIHRoZSBwcmV0dHktcHJp bnRlcnMuCi1mcm9tIGxpYnN0ZGN4eC52Ni5wcmludGVycyBpbXBvcnQgcmVn aXN0ZXJfbGlic3RkY3h4X3ByaW50ZXJzCi1yZWdpc3Rlcl9saWJzdGRjeHhf cHJpbnRlcnMgKGdkYi5jdXJyZW50X29iamZpbGUgKCkpCi0KLSMgTG9hZCB0 aGUgeG1ldGhvZHMgaWYgR0RCIHN1cHBvcnRzIHRoZW0uCi1kZWYgZ2RiX2hh c194bWV0aG9kcygpOgotICAgIHRyeToKLSAgICAgICAgaW1wb3J0IGdkYi54 bWV0aG9kCi0gICAgICAgIHJldHVybiBUcnVlCi0gICAgZXhjZXB0IEltcG9y dEVycm9yOgotICAgICAgICByZXR1cm4gRmFsc2UKLQotaWYgZ2RiX2hhc194 bWV0aG9kcygpOgotICAgIGZyb20gbGlic3RkY3h4LnY2LnhtZXRob2RzIGlt cG9ydCByZWdpc3Rlcl9saWJzdGRjeHhfeG1ldGhvZHMKLSAgICByZWdpc3Rl cl9saWJzdGRjeHhfeG1ldGhvZHMgKGdkYi5jdXJyZW50X29iamZpbGUgKCkp CitpbXBvcnQgbGlic3RkY3h4LnY2CmRpZmYgLS1naXQgYS9saWJzdGRjKyst djMvcHl0aG9uL2xpYnN0ZGN4eC92Ni9fX2luaXRfXy5weSBiL2xpYnN0ZGMr Ky12My9weXRob24vbGlic3RkY3h4L3Y2L19faW5pdF9fLnB5CmluZGV4IDhi MTM3ODkuLjU5YzFmMjcgMTAwNjQ0Ci0tLSBhL2xpYnN0ZGMrKy12My9weXRo b24vbGlic3RkY3h4L3Y2L19faW5pdF9fLnB5CisrKyBiL2xpYnN0ZGMrKy12 My9weXRob24vbGlic3RkY3h4L3Y2L19faW5pdF9fLnB5CkBAIC0xICsxLDMy IEBACisjIENvcHlyaWdodCAoQykgMjAxNCBGcmVlIFNvZnR3YXJlIEZvdW5k YXRpb24sIEluYy4KIAorIyBUaGlzIHByb2dyYW0gaXMgZnJlZSBzb2Z0d2Fy ZTsgeW91IGNhbiByZWRpc3RyaWJ1dGUgaXQgYW5kL29yIG1vZGlmeQorIyBp dCB1bmRlciB0aGUgdGVybXMgb2YgdGhlIEdOVSBHZW5lcmFsIFB1YmxpYyBM aWNlbnNlIGFzIHB1Ymxpc2hlZCBieQorIyB0aGUgRnJlZSBTb2Z0d2FyZSBG b3VuZGF0aW9uOyBlaXRoZXIgdmVyc2lvbiAzIG9mIHRoZSBMaWNlbnNlLCBv cgorIyAoYXQgeW91ciBvcHRpb24pIGFueSBsYXRlciB2ZXJzaW9uLgorIwor IyBUaGlzIHByb2dyYW0gaXMgZGlzdHJpYnV0ZWQgaW4gdGhlIGhvcGUgdGhh dCBpdCB3aWxsIGJlIHVzZWZ1bCwKKyMgYnV0IFdJVEhPVVQgQU5ZIFdBUlJB TlRZOyB3aXRob3V0IGV2ZW4gdGhlIGltcGxpZWQgd2FycmFudHkgb2YKKyMg TUVSQ0hBTlRBQklMSVRZIG9yIEZJVE5FU1MgRk9SIEEgUEFSVElDVUxBUiBQ VVJQT1NFLiAgU2VlIHRoZQorIyBHTlUgR2VuZXJhbCBQdWJsaWMgTGljZW5z ZSBmb3IgbW9yZSBkZXRhaWxzLgorIworIyBZb3Ugc2hvdWxkIGhhdmUgcmVj ZWl2ZWQgYSBjb3B5IG9mIHRoZSBHTlUgR2VuZXJhbCBQdWJsaWMgTGljZW5z ZQorIyBhbG9uZyB3aXRoIHRoaXMgcHJvZ3JhbS4gIElmIG5vdCwgc2VlIDxo dHRwOi8vd3d3LmdudS5vcmcvbGljZW5zZXMvPi4KKworaW1wb3J0IGdkYgor CisjIExvYWQgdGhlIHByZXR0eS1wcmludGVycy4KK2Zyb20gcHJpbnRlcnMg aW1wb3J0IHJlZ2lzdGVyX2xpYnN0ZGN4eF9wcmludGVycworcmVnaXN0ZXJf bGlic3RkY3h4X3ByaW50ZXJzKGdkYi5jdXJyZW50X29iamZpbGUoKSkKKwor IyBMb2FkIHRoZSB4bWV0aG9kcyBpZiBHREIgc3VwcG9ydHMgdGhlbS4KK2Rl ZiBnZGJfaGFzX3htZXRob2RzKCk6CisgICAgdHJ5OgorICAgICAgICBpbXBv cnQgZ2RiLnhtZXRob2QKKyAgICAgICAgcmV0dXJuIFRydWUKKyAgICBleGNl cHQgSW1wb3J0RXJyb3I6CisgICAgICAgIHJldHVybiBGYWxzZQorCitpZiBn ZGJfaGFzX3htZXRob2RzKCk6CisgICAgZnJvbSB4bWV0aG9kcyBpbXBvcnQg cmVnaXN0ZXJfbGlic3RkY3h4X3htZXRob2RzCisgICAgcmVnaXN0ZXJfbGli c3RkY3h4X3htZXRob2RzKGdkYi5jdXJyZW50X29iamZpbGUoKSkK --e89a8f83a883cdecdc050433e156--