U++ forum: Welcome to the forum

Status & Roadmap

Authors & License

Funding Ultimate++

Search on this site

Search in forums

Home » U++ Library support » U++ Core » Vector<T>::Set(int i, T&& x) proposal

Show: Today's Messages :: Show Polls :: Message Navigator
E-mail to friend

Return to the default flat view

Create a new topic

Submit Reply

Re: Vector<T>::Set(int i, T&& x) proposal [message #49345 is a reply to message #49336]

Tue, 30 January 2018 03:30

Novo is currently offline

Novo
Messages: 1358
Registered: December 2006

Ultimate Contributor

mirek wrote on Sun, 28 January 2018 16:47

I am considering this, but single element Set was always just v.At(i) = pick(src);

Well, yes. This will work as well. But this is significantly less intuitive. Your way of solving this problem definitely didn't come to my mind when I was looking for a solution. Adding of an rvalue-based version of Set definitely won't break API because you already have a reference-based version. This is just another performance optimization.

My solution is based on move-constructor and yours is based on move-operator.

Regards,
Novo

[Updated on: Tue, 30 January 2018 03:45]

Report message to a moderator

Send a private message to this user

[Message index]

		Vector<T>::Set(int i, T&& x) proposal By: Novo on Tue, 19 December 2017 23:30
		Re: Vector<T>::Set(int i, T&& x) proposal By: mirek on Sun, 28 January 2018 22:47
		Re: Vector<T>::Set(int i, T&& x) proposal By: Novo on Tue, 30 January 2018 03:30
		Re: Vector<T>::Set(int i, T&& x) proposal By: mirek on Wed, 31 January 2018 19:34
		Re: Vector<T>::Set(int i, T&& x) proposal By: Novo on Wed, 31 January 2018 20:01
		Re: Vector<T>::Set(int i, T&& x) proposal By: mirek on Thu, 01 February 2018 18:53
		Re: Vector<T>::Set(int i, T&& x) proposal By: Novo on Thu, 01 February 2018 19:03

Previous Topic:	Function.h: "expression cannot be used as a function"
Next Topic:	Core/SSL crashes at exit on Linux.

Goto Forum:

-=] Back to Top [=-

[ Syndicate this forum (XML) ] [

] [

PDF

]

Current Time: Tue May 21 18:30:06 CEST 2024

Total time taken to generate the page: 0.01887 seconds