Should a using declaration be able to refer to a friend function












2















I have a class with a friend function defined in the class head. When I do a using declaration of this friend function it works in MSVC2017 but neither in GCC 8.2 or Clang 7.0. Who is right? Here is the godbolt link (https://godbolt.org/z/_7MVlh), and here is the code:



namespace vec {

class Vec {
friend Vec vec_max(const Vec& a, const Vec& b) { return a; }
};

}

void test() {
using vec::vec_max;
}


GCC gives the error on the using declaration: "error: no member named 'vec_max' in namespace 'vec'". Clang gave a similar message. MSVC compiled it as intended.



According to my understanding vec_max should reside in the vec namespace and MSVC should be correct. But there may be some subtle writing in the standard that makes the more restrictive interpretation of gcc and clang correct (although less intuitive).










share|improve this question


















  • 3





    friend functions defined inline in a class can only be found through ADL for that class, so MSVC is wrong here.

    – Quentin
    Nov 21 '18 at 10:15


















2















I have a class with a friend function defined in the class head. When I do a using declaration of this friend function it works in MSVC2017 but neither in GCC 8.2 or Clang 7.0. Who is right? Here is the godbolt link (https://godbolt.org/z/_7MVlh), and here is the code:



namespace vec {

class Vec {
friend Vec vec_max(const Vec& a, const Vec& b) { return a; }
};

}

void test() {
using vec::vec_max;
}


GCC gives the error on the using declaration: "error: no member named 'vec_max' in namespace 'vec'". Clang gave a similar message. MSVC compiled it as intended.



According to my understanding vec_max should reside in the vec namespace and MSVC should be correct. But there may be some subtle writing in the standard that makes the more restrictive interpretation of gcc and clang correct (although less intuitive).










share|improve this question


















  • 3





    friend functions defined inline in a class can only be found through ADL for that class, so MSVC is wrong here.

    – Quentin
    Nov 21 '18 at 10:15
















2












2








2








I have a class with a friend function defined in the class head. When I do a using declaration of this friend function it works in MSVC2017 but neither in GCC 8.2 or Clang 7.0. Who is right? Here is the godbolt link (https://godbolt.org/z/_7MVlh), and here is the code:



namespace vec {

class Vec {
friend Vec vec_max(const Vec& a, const Vec& b) { return a; }
};

}

void test() {
using vec::vec_max;
}


GCC gives the error on the using declaration: "error: no member named 'vec_max' in namespace 'vec'". Clang gave a similar message. MSVC compiled it as intended.



According to my understanding vec_max should reside in the vec namespace and MSVC should be correct. But there may be some subtle writing in the standard that makes the more restrictive interpretation of gcc and clang correct (although less intuitive).










share|improve this question














I have a class with a friend function defined in the class head. When I do a using declaration of this friend function it works in MSVC2017 but neither in GCC 8.2 or Clang 7.0. Who is right? Here is the godbolt link (https://godbolt.org/z/_7MVlh), and here is the code:



namespace vec {

class Vec {
friend Vec vec_max(const Vec& a, const Vec& b) { return a; }
};

}

void test() {
using vec::vec_max;
}


GCC gives the error on the using declaration: "error: no member named 'vec_max' in namespace 'vec'". Clang gave a similar message. MSVC compiled it as intended.



According to my understanding vec_max should reside in the vec namespace and MSVC should be correct. But there may be some subtle writing in the standard that makes the more restrictive interpretation of gcc and clang correct (although less intuitive).







c++ declaration using friend






share|improve this question













share|improve this question











share|improve this question




share|improve this question










asked Nov 21 '18 at 10:11









Bengt GustafssonBengt Gustafsson

19428




19428








  • 3





    friend functions defined inline in a class can only be found through ADL for that class, so MSVC is wrong here.

    – Quentin
    Nov 21 '18 at 10:15
















  • 3





    friend functions defined inline in a class can only be found through ADL for that class, so MSVC is wrong here.

    – Quentin
    Nov 21 '18 at 10:15










3




3





friend functions defined inline in a class can only be found through ADL for that class, so MSVC is wrong here.

– Quentin
Nov 21 '18 at 10:15







friend functions defined inline in a class can only be found through ADL for that class, so MSVC is wrong here.

– Quentin
Nov 21 '18 at 10:15














1 Answer
1






active

oldest

votes


















3














MSVC is wrong here. When a friend function is defined inline within a class definition and not declared outside, it should not be found by normal name lookup and only Argument-Dependent Lookup should be able to find it.



Quoting C++17 (n4659):



14.3 [class.friend]




6 A function can be defined in a friend declaration of a class if and only if the class is a non-local class (12.4),
the function name is unqualified, and the function has namespace scope. ...



7 Such a function is implicitly an inline function (10.1.6). A friend function defined in a class is in the (lexical)
scope of the class in which it is defined. A friend function defined outside the class is not (6.4.1).




and 6.4.2/4 [basic.lookup.argdep]




When considering an associated namespace, the lookup is the same as the lookup performed when the
associated namespace is used as a qualifier (6.4.3.2) except that:




  • (4.1) ...


  • (4.2) Any namespace-scope friend functions or friend function templates declared in associated classes are
    visible within their respective namespaces even if they are not visible during an ordinary lookup (14.3).


  • (4.3) ...







share|improve this answer

























    Your Answer






    StackExchange.ifUsing("editor", function () {
    StackExchange.using("externalEditor", function () {
    StackExchange.using("snippets", function () {
    StackExchange.snippets.init();
    });
    });
    }, "code-snippets");

    StackExchange.ready(function() {
    var channelOptions = {
    tags: "".split(" "),
    id: "1"
    };
    initTagRenderer("".split(" "), "".split(" "), channelOptions);

    StackExchange.using("externalEditor", function() {
    // Have to fire editor after snippets, if snippets enabled
    if (StackExchange.settings.snippets.snippetsEnabled) {
    StackExchange.using("snippets", function() {
    createEditor();
    });
    }
    else {
    createEditor();
    }
    });

    function createEditor() {
    StackExchange.prepareEditor({
    heartbeatType: 'answer',
    autoActivateHeartbeat: false,
    convertImagesToLinks: true,
    noModals: true,
    showLowRepImageUploadWarning: true,
    reputationToPostImages: 10,
    bindNavPrevention: true,
    postfix: "",
    imageUploader: {
    brandingHtml: "Powered by u003ca class="icon-imgur-white" href="https://imgur.com/"u003eu003c/au003e",
    contentPolicyHtml: "User contributions licensed under u003ca href="https://creativecommons.org/licenses/by-sa/3.0/"u003ecc by-sa 3.0 with attribution requiredu003c/au003e u003ca href="https://stackoverflow.com/legal/content-policy"u003e(content policy)u003c/au003e",
    allowUrls: true
    },
    onDemand: true,
    discardSelector: ".discard-answer"
    ,immediatelyShowMarkdownHelp:true
    });


    }
    });














    draft saved

    draft discarded


















    StackExchange.ready(
    function () {
    StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fstackoverflow.com%2fquestions%2f53409706%2fshould-a-using-declaration-be-able-to-refer-to-a-friend-function%23new-answer', 'question_page');
    }
    );

    Post as a guest















    Required, but never shown

























    1 Answer
    1






    active

    oldest

    votes








    1 Answer
    1






    active

    oldest

    votes









    active

    oldest

    votes






    active

    oldest

    votes









    3














    MSVC is wrong here. When a friend function is defined inline within a class definition and not declared outside, it should not be found by normal name lookup and only Argument-Dependent Lookup should be able to find it.



    Quoting C++17 (n4659):



    14.3 [class.friend]




    6 A function can be defined in a friend declaration of a class if and only if the class is a non-local class (12.4),
    the function name is unqualified, and the function has namespace scope. ...



    7 Such a function is implicitly an inline function (10.1.6). A friend function defined in a class is in the (lexical)
    scope of the class in which it is defined. A friend function defined outside the class is not (6.4.1).




    and 6.4.2/4 [basic.lookup.argdep]




    When considering an associated namespace, the lookup is the same as the lookup performed when the
    associated namespace is used as a qualifier (6.4.3.2) except that:




    • (4.1) ...


    • (4.2) Any namespace-scope friend functions or friend function templates declared in associated classes are
      visible within their respective namespaces even if they are not visible during an ordinary lookup (14.3).


    • (4.3) ...







    share|improve this answer






























      3














      MSVC is wrong here. When a friend function is defined inline within a class definition and not declared outside, it should not be found by normal name lookup and only Argument-Dependent Lookup should be able to find it.



      Quoting C++17 (n4659):



      14.3 [class.friend]




      6 A function can be defined in a friend declaration of a class if and only if the class is a non-local class (12.4),
      the function name is unqualified, and the function has namespace scope. ...



      7 Such a function is implicitly an inline function (10.1.6). A friend function defined in a class is in the (lexical)
      scope of the class in which it is defined. A friend function defined outside the class is not (6.4.1).




      and 6.4.2/4 [basic.lookup.argdep]




      When considering an associated namespace, the lookup is the same as the lookup performed when the
      associated namespace is used as a qualifier (6.4.3.2) except that:




      • (4.1) ...


      • (4.2) Any namespace-scope friend functions or friend function templates declared in associated classes are
        visible within their respective namespaces even if they are not visible during an ordinary lookup (14.3).


      • (4.3) ...







      share|improve this answer




























        3












        3








        3







        MSVC is wrong here. When a friend function is defined inline within a class definition and not declared outside, it should not be found by normal name lookup and only Argument-Dependent Lookup should be able to find it.



        Quoting C++17 (n4659):



        14.3 [class.friend]




        6 A function can be defined in a friend declaration of a class if and only if the class is a non-local class (12.4),
        the function name is unqualified, and the function has namespace scope. ...



        7 Such a function is implicitly an inline function (10.1.6). A friend function defined in a class is in the (lexical)
        scope of the class in which it is defined. A friend function defined outside the class is not (6.4.1).




        and 6.4.2/4 [basic.lookup.argdep]




        When considering an associated namespace, the lookup is the same as the lookup performed when the
        associated namespace is used as a qualifier (6.4.3.2) except that:




        • (4.1) ...


        • (4.2) Any namespace-scope friend functions or friend function templates declared in associated classes are
          visible within their respective namespaces even if they are not visible during an ordinary lookup (14.3).


        • (4.3) ...







        share|improve this answer















        MSVC is wrong here. When a friend function is defined inline within a class definition and not declared outside, it should not be found by normal name lookup and only Argument-Dependent Lookup should be able to find it.



        Quoting C++17 (n4659):



        14.3 [class.friend]




        6 A function can be defined in a friend declaration of a class if and only if the class is a non-local class (12.4),
        the function name is unqualified, and the function has namespace scope. ...



        7 Such a function is implicitly an inline function (10.1.6). A friend function defined in a class is in the (lexical)
        scope of the class in which it is defined. A friend function defined outside the class is not (6.4.1).




        and 6.4.2/4 [basic.lookup.argdep]




        When considering an associated namespace, the lookup is the same as the lookup performed when the
        associated namespace is used as a qualifier (6.4.3.2) except that:




        • (4.1) ...


        • (4.2) Any namespace-scope friend functions or friend function templates declared in associated classes are
          visible within their respective namespaces even if they are not visible during an ordinary lookup (14.3).


        • (4.3) ...








        share|improve this answer














        share|improve this answer



        share|improve this answer








        edited Nov 21 '18 at 10:22

























        answered Nov 21 '18 at 10:16









        AngewAngew

        133k11258350




        133k11258350
































            draft saved

            draft discarded




















































            Thanks for contributing an answer to Stack Overflow!


            • Please be sure to answer the question. Provide details and share your research!

            But avoid



            • Asking for help, clarification, or responding to other answers.

            • Making statements based on opinion; back them up with references or personal experience.


            To learn more, see our tips on writing great answers.




            draft saved


            draft discarded














            StackExchange.ready(
            function () {
            StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fstackoverflow.com%2fquestions%2f53409706%2fshould-a-using-declaration-be-able-to-refer-to-a-friend-function%23new-answer', 'question_page');
            }
            );

            Post as a guest















            Required, but never shown





















































            Required, but never shown














            Required, but never shown












            Required, but never shown







            Required, but never shown

































            Required, but never shown














            Required, but never shown












            Required, but never shown







            Required, but never shown







            Popular posts from this blog

            Guess what letter conforming each word

            Port of Spain

            Run scheduled task as local user group (not BUILTIN)