Compiler doesn't fail when pushing back a std::unique_ptr into a std::vectorConcatenating two std::vectorsHow to find out if an item is present in a std::vector?What is std::move(), and when should it be used?std::auto_ptr to std::unique_ptrHow to store a vector of objects of an abstract class which are given by std::unique_ptr?C++ std::unique_ptr but non movable?Move objects from a unique_ptr array to a vectorVector push_back error when compilingpush_back vs emplace_back to a std::vector<std::string>Move std::unique_ptr<int[]> to Platform::Collections::Vector<int>

Bold and Colored Image Caption

Did depressed people far more accurately estimate how many monsters they killed in a video game?

Can a landlord force all residents to use the landlord's in-house debit card accounts?

How to properly translate the key phrase of Erdoğan's 2016 letter to Putin, "kusura bakmasınlar," to Russian

What would +1/+2/+3 items be called in game?

Why does Trump want a citizenship question on the census?

Hail hit my roof. Do I need to replace it?

Why is a mixture of two normally distributed variables only bimodal if their means differ by at least two times the common standard deviation?

Found and corrected a mistake on someone's else paper -- praxis?

Compiler doesn't fail when pushing back a std::unique_ptr into a std::vector

How does the Melf's Minute Meteors spell interact with the Evocation wizard's Sculpt Spells feature?

Why does the Antonov AN-225 not have any winglets?

How do you move up one folder in Finder?

How do I explain that I don't want to maintain old projects?

What are the effects of abstaining from eating a certain flavor?

If props is missing Should I use memo?

Intern not wearing safety equipment; how could I have handled this differently?

Users forgetting to regenerate PDF before sending it

What does the multimeter dial do internally?

Is there a way I can open the Windows 10 Ubuntu bash without running the ~/.bashrc script?

Is it okay to use open source code to do an interview task?

When did "&" stop being taught alongside the alphabet?

A Logic Puzzle—One of These Things Doesn’t Quite Fit

Swapping "Good" and "Bad"



Compiler doesn't fail when pushing back a std::unique_ptr into a std::vector


Concatenating two std::vectorsHow to find out if an item is present in a std::vector?What is std::move(), and when should it be used?std::auto_ptr to std::unique_ptrHow to store a vector of objects of an abstract class which are given by std::unique_ptr?C++ std::unique_ptr but non movable?Move objects from a unique_ptr array to a vectorVector push_back error when compilingpush_back vs emplace_back to a std::vector<std::string>Move std::unique_ptr<int[]> to Platform::Collections::Vector<int>






.everyoneloves__top-leaderboard:empty,.everyoneloves__mid-leaderboard:empty,.everyoneloves__bot-mid-leaderboard:empty margin-bottom:0;








6















An unique_ptr cannot be pushed back into a std::vector since it is non-copyable, unless std::move is used. However, let be F a function that returns an unique_ptr, then, the operation std::vector::push_back(F()) is allowed. There is an example below:



#include <iostream>
#include <vector>
#include <memory>

class A
public:
int f() return _f + 10;

private:
int _f = 20;
;

std::unique_ptr<A> create() return std::unique_ptr<A>(new A);


int main()
std::unique_ptr<A> p1(new A());

std::vector< std::unique_ptr<A> > v;

v.push_back(p1); // (1) This fails, should use std::move

v.push_back(create()); // (2) This doesn't fail, should use std::move?

return 0;



(2) is allowed, but (1) is not. Is this because the returned value is moved somehow implicitly?



In (2), is it actually necessary to use std::move?










share|improve this question






























    6















    An unique_ptr cannot be pushed back into a std::vector since it is non-copyable, unless std::move is used. However, let be F a function that returns an unique_ptr, then, the operation std::vector::push_back(F()) is allowed. There is an example below:



    #include <iostream>
    #include <vector>
    #include <memory>

    class A
    public:
    int f() return _f + 10;

    private:
    int _f = 20;
    ;

    std::unique_ptr<A> create() return std::unique_ptr<A>(new A);


    int main()
    std::unique_ptr<A> p1(new A());

    std::vector< std::unique_ptr<A> > v;

    v.push_back(p1); // (1) This fails, should use std::move

    v.push_back(create()); // (2) This doesn't fail, should use std::move?

    return 0;



    (2) is allowed, but (1) is not. Is this because the returned value is moved somehow implicitly?



    In (2), is it actually necessary to use std::move?










    share|improve this question


























      6












      6








      6








      An unique_ptr cannot be pushed back into a std::vector since it is non-copyable, unless std::move is used. However, let be F a function that returns an unique_ptr, then, the operation std::vector::push_back(F()) is allowed. There is an example below:



      #include <iostream>
      #include <vector>
      #include <memory>

      class A
      public:
      int f() return _f + 10;

      private:
      int _f = 20;
      ;

      std::unique_ptr<A> create() return std::unique_ptr<A>(new A);


      int main()
      std::unique_ptr<A> p1(new A());

      std::vector< std::unique_ptr<A> > v;

      v.push_back(p1); // (1) This fails, should use std::move

      v.push_back(create()); // (2) This doesn't fail, should use std::move?

      return 0;



      (2) is allowed, but (1) is not. Is this because the returned value is moved somehow implicitly?



      In (2), is it actually necessary to use std::move?










      share|improve this question
















      An unique_ptr cannot be pushed back into a std::vector since it is non-copyable, unless std::move is used. However, let be F a function that returns an unique_ptr, then, the operation std::vector::push_back(F()) is allowed. There is an example below:



      #include <iostream>
      #include <vector>
      #include <memory>

      class A
      public:
      int f() return _f + 10;

      private:
      int _f = 20;
      ;

      std::unique_ptr<A> create() return std::unique_ptr<A>(new A);


      int main()
      std::unique_ptr<A> p1(new A());

      std::vector< std::unique_ptr<A> > v;

      v.push_back(p1); // (1) This fails, should use std::move

      v.push_back(create()); // (2) This doesn't fail, should use std::move?

      return 0;



      (2) is allowed, but (1) is not. Is this because the returned value is moved somehow implicitly?



      In (2), is it actually necessary to use std::move?







      c++ smart-pointers stdvector move-semantics






      share|improve this question















      share|improve this question













      share|improve this question




      share|improve this question








      edited 9 hours ago









      0x499602D2

      70.2k29 gold badges124 silver badges211 bronze badges




      70.2k29 gold badges124 silver badges211 bronze badges










      asked 9 hours ago









      DanDan

      8287 silver badges21 bronze badges




      8287 silver badges21 bronze badges






















          3 Answers
          3






          active

          oldest

          votes


















          11














          std::move(X) essentially means "here, treat X as if it was a temporary object".



          create() returns a temporary std::unique_ptr<A> to begin with, so move is unnecessary.




          If you want to know more, look into the value categories. Your compiler uses value categories to determine if an expression refers to a temporary object ("rvalue") or not ("lvalue").



          p1 is an lvalue, and create() is an rvalue.






          share|improve this answer
































            2














            std::vector::push_back() has an overload that takes an rvalue reference as input:



            void push_back( T&& value );


            The return value of create() is an unnamed temporary, ie an rvalue, so it can be passed as-is to push_back() without needing to use std::move() on it.



            std::move() is needed only when passing a named variable, ie an lvalue, where an rvalue is expected.






            share|improve this answer






























              2














              With C++11 we got move constructors and rvalues semantics.



              std::move(X) is just a cast to a rvalue which converts X to X&& that is it. Than move ctor takes the job over and move constructors typically "steal" the resources held by the argument. unique_ptr have a move ctor.



              Function return values are already a rvalue(unless the function returns an lvalue reference as indicated by @HolyBlackCat in comments) which will trigger the move ctor without needing any extra cast. And since move ctor is defined for unique_ptr it will compile.



              Also the reason why v.push_back(p1);failing is: you try to call copy constructor with an lvalue and it fails because unique_ptr does not have a copy ctor.






              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%2f56925919%2fcompiler-doesnt-fail-when-pushing-back-a-stdunique-ptr-into-a-stdvector%23new-answer', 'question_page');

                );

                Post as a guest















                Required, but never shown

























                3 Answers
                3






                active

                oldest

                votes








                3 Answers
                3






                active

                oldest

                votes









                active

                oldest

                votes






                active

                oldest

                votes









                11














                std::move(X) essentially means "here, treat X as if it was a temporary object".



                create() returns a temporary std::unique_ptr<A> to begin with, so move is unnecessary.




                If you want to know more, look into the value categories. Your compiler uses value categories to determine if an expression refers to a temporary object ("rvalue") or not ("lvalue").



                p1 is an lvalue, and create() is an rvalue.






                share|improve this answer





























                  11














                  std::move(X) essentially means "here, treat X as if it was a temporary object".



                  create() returns a temporary std::unique_ptr<A> to begin with, so move is unnecessary.




                  If you want to know more, look into the value categories. Your compiler uses value categories to determine if an expression refers to a temporary object ("rvalue") or not ("lvalue").



                  p1 is an lvalue, and create() is an rvalue.






                  share|improve this answer



























                    11












                    11








                    11







                    std::move(X) essentially means "here, treat X as if it was a temporary object".



                    create() returns a temporary std::unique_ptr<A> to begin with, so move is unnecessary.




                    If you want to know more, look into the value categories. Your compiler uses value categories to determine if an expression refers to a temporary object ("rvalue") or not ("lvalue").



                    p1 is an lvalue, and create() is an rvalue.






                    share|improve this answer















                    std::move(X) essentially means "here, treat X as if it was a temporary object".



                    create() returns a temporary std::unique_ptr<A> to begin with, so move is unnecessary.




                    If you want to know more, look into the value categories. Your compiler uses value categories to determine if an expression refers to a temporary object ("rvalue") or not ("lvalue").



                    p1 is an lvalue, and create() is an rvalue.







                    share|improve this answer














                    share|improve this answer



                    share|improve this answer








                    edited 9 hours ago

























                    answered 9 hours ago









                    HolyBlackCatHolyBlackCat

                    19.4k4 gold badges40 silver badges73 bronze badges




                    19.4k4 gold badges40 silver badges73 bronze badges























                        2














                        std::vector::push_back() has an overload that takes an rvalue reference as input:



                        void push_back( T&& value );


                        The return value of create() is an unnamed temporary, ie an rvalue, so it can be passed as-is to push_back() without needing to use std::move() on it.



                        std::move() is needed only when passing a named variable, ie an lvalue, where an rvalue is expected.






                        share|improve this answer



























                          2














                          std::vector::push_back() has an overload that takes an rvalue reference as input:



                          void push_back( T&& value );


                          The return value of create() is an unnamed temporary, ie an rvalue, so it can be passed as-is to push_back() without needing to use std::move() on it.



                          std::move() is needed only when passing a named variable, ie an lvalue, where an rvalue is expected.






                          share|improve this answer

























                            2












                            2








                            2







                            std::vector::push_back() has an overload that takes an rvalue reference as input:



                            void push_back( T&& value );


                            The return value of create() is an unnamed temporary, ie an rvalue, so it can be passed as-is to push_back() without needing to use std::move() on it.



                            std::move() is needed only when passing a named variable, ie an lvalue, where an rvalue is expected.






                            share|improve this answer













                            std::vector::push_back() has an overload that takes an rvalue reference as input:



                            void push_back( T&& value );


                            The return value of create() is an unnamed temporary, ie an rvalue, so it can be passed as-is to push_back() without needing to use std::move() on it.



                            std::move() is needed only when passing a named variable, ie an lvalue, where an rvalue is expected.







                            share|improve this answer












                            share|improve this answer



                            share|improve this answer










                            answered 9 hours ago









                            Remy LebeauRemy Lebeau

                            354k19 gold badges286 silver badges480 bronze badges




                            354k19 gold badges286 silver badges480 bronze badges





















                                2














                                With C++11 we got move constructors and rvalues semantics.



                                std::move(X) is just a cast to a rvalue which converts X to X&& that is it. Than move ctor takes the job over and move constructors typically "steal" the resources held by the argument. unique_ptr have a move ctor.



                                Function return values are already a rvalue(unless the function returns an lvalue reference as indicated by @HolyBlackCat in comments) which will trigger the move ctor without needing any extra cast. And since move ctor is defined for unique_ptr it will compile.



                                Also the reason why v.push_back(p1);failing is: you try to call copy constructor with an lvalue and it fails because unique_ptr does not have a copy ctor.






                                share|improve this answer





























                                  2














                                  With C++11 we got move constructors and rvalues semantics.



                                  std::move(X) is just a cast to a rvalue which converts X to X&& that is it. Than move ctor takes the job over and move constructors typically "steal" the resources held by the argument. unique_ptr have a move ctor.



                                  Function return values are already a rvalue(unless the function returns an lvalue reference as indicated by @HolyBlackCat in comments) which will trigger the move ctor without needing any extra cast. And since move ctor is defined for unique_ptr it will compile.



                                  Also the reason why v.push_back(p1);failing is: you try to call copy constructor with an lvalue and it fails because unique_ptr does not have a copy ctor.






                                  share|improve this answer



























                                    2












                                    2








                                    2







                                    With C++11 we got move constructors and rvalues semantics.



                                    std::move(X) is just a cast to a rvalue which converts X to X&& that is it. Than move ctor takes the job over and move constructors typically "steal" the resources held by the argument. unique_ptr have a move ctor.



                                    Function return values are already a rvalue(unless the function returns an lvalue reference as indicated by @HolyBlackCat in comments) which will trigger the move ctor without needing any extra cast. And since move ctor is defined for unique_ptr it will compile.



                                    Also the reason why v.push_back(p1);failing is: you try to call copy constructor with an lvalue and it fails because unique_ptr does not have a copy ctor.






                                    share|improve this answer















                                    With C++11 we got move constructors and rvalues semantics.



                                    std::move(X) is just a cast to a rvalue which converts X to X&& that is it. Than move ctor takes the job over and move constructors typically "steal" the resources held by the argument. unique_ptr have a move ctor.



                                    Function return values are already a rvalue(unless the function returns an lvalue reference as indicated by @HolyBlackCat in comments) which will trigger the move ctor without needing any extra cast. And since move ctor is defined for unique_ptr it will compile.



                                    Also the reason why v.push_back(p1);failing is: you try to call copy constructor with an lvalue and it fails because unique_ptr does not have a copy ctor.







                                    share|improve this answer














                                    share|improve this answer



                                    share|improve this answer








                                    edited 8 hours ago

























                                    answered 9 hours ago









                                    Kadir Erdem DemirKadir Erdem Demir

                                    2,2972 gold badges20 silver badges32 bronze badges




                                    2,2972 gold badges20 silver badges32 bronze badges



























                                        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%2f56925919%2fcompiler-doesnt-fail-when-pushing-back-a-stdunique-ptr-into-a-stdvector%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

                                        Invision Community Contents History See also References External links Navigation menuProprietaryinvisioncommunity.comIPS Community ForumsIPS Community Forumsthis blog entry"License Changes, IP.Board 3.4, and the Future""Interview -- Matt Mecham of Ibforums""CEO Invision Power Board, Matt Mecham Is a Liar, Thief!"IPB License Explanation 1.3, 1.3.1, 2.0, and 2.1ArchivedSecurity Fixes, Updates And Enhancements For IPB 1.3.1Archived"New Demo Accounts - Invision Power Services"the original"New Default Skin"the original"Invision Power Board 3.0.0 and Applications Released"the original"Archived copy"the original"Perpetual licenses being done away with""Release Notes - Invision Power Services""Introducing: IPS Community Suite 4!"Invision Community Release Notes

                                        Canceling a color specificationRandomly assigning color to Graphics3D objects?Default color for Filling in Mathematica 9Coloring specific elements of sets with a prime modified order in an array plotHow to pick a color differing significantly from the colors already in a given color list?Detection of the text colorColor numbers based on their valueCan color schemes for use with ColorData include opacity specification?My dynamic color schemes

                                        Tom Holland Mục lục Đầu đời và giáo dục | Sự nghiệp | Cuộc sống cá nhân | Phim tham gia | Giải thưởng và đề cử | Chú thích | Liên kết ngoài | Trình đơn chuyển hướngProfile“Person Details for Thomas Stanley Holland, "England and Wales Birth Registration Index, 1837-2008" — FamilySearch.org”"Meet Tom Holland... the 16-year-old star of The Impossible""Schoolboy actor Tom Holland finds himself in Oscar contention for role in tsunami drama"“Naomi Watts on the Prince William and Harry's reaction to her film about the late Princess Diana”lưu trữ"Holland and Pflueger Are West End's Two New 'Billy Elliots'""I'm so envious of my son, the movie star! British writer Dominic Holland's spent 20 years trying to crack Hollywood - but he's been beaten to it by a very unlikely rival"“Richard and Margaret Povey of Jersey, Channel Islands, UK: Information about Thomas Stanley Holland”"Tom Holland to play Billy Elliot""New Billy Elliot leaving the garage"Billy Elliot the Musical - Tom Holland - Billy"A Tale of four Billys: Tom Holland""The Feel Good Factor""Thames Christian College schoolboys join Myleene Klass for The Feelgood Factor""Government launches £600,000 arts bursaries pilot""BILLY's Chapman, Holland, Gardner & Jackson-Keen Visit Prime Minister""Elton John 'blown away' by Billy Elliot fifth birthday" (video with John's interview and fragments of Holland's performance)"First News interviews Arrietty's Tom Holland"“33rd Critics' Circle Film Awards winners”“National Board of Review Current Awards”Bản gốc"Ron Howard Whaling Tale 'In The Heart Of The Sea' Casts Tom Holland"“'Spider-Man' Finds Tom Holland to Star as New Web-Slinger”lưu trữ“Captain America: Civil War (2016)”“Film Review: ‘Captain America: Civil War’”lưu trữ“‘Captain America: Civil War’ review: Choose your own avenger”lưu trữ“The Lost City of Z reviews”“Sony Pictures and Marvel Studios Find Their 'Spider-Man' Star and Director”“‘Mary Magdalene’, ‘Current War’ & ‘Wind River’ Get 2017 Release Dates From Weinstein”“Lionsgate Unleashing Daisy Ridley & Tom Holland Starrer ‘Chaos Walking’ In Cannes”“PTA's 'Master' Leads Chicago Film Critics Nominations, UPDATED: Houston and Indiana Critics Nominations”“Nominaciones Goya 2013 Telecinco Cinema – ENG”“Jameson Empire Film Awards: Martin Freeman wins best actor for performance in The Hobbit”“34th Annual Young Artist Awards”Bản gốc“Teen Choice Awards 2016—Captain America: Civil War Leads Second Wave of Nominations”“BAFTA Film Award Nominations: ‘La La Land’ Leads Race”“Saturn Awards Nominations 2017: 'Rogue One,' 'Walking Dead' Lead”Tom HollandTom HollandTom HollandTom Hollandmedia.gettyimages.comWorldCat Identities300279794no20130442900000 0004 0355 42791085670554170004732cb16706349t(data)XX5557367