U++ forum: Welcome to the forum

Status & Roadmap

Authors & License

Funding Ultimate++

Search on this site

Search in forums

Home » Community » U++ community news and announcements » get_i

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: get_i [message #54353 is a reply to message #54350]

Fri, 03 July 2020 09:38

mirek is currently offline

mirek
Messages: 14256
Registered: November 2005

Ultimate Member

Novo wrote on Thu, 02 July 2020 22:08

Sorry for the late response.
My code is correct. Temporary String lives only during function call. This is how C++ works.

I would not be fixing if it was correct. Whole thing was actual error in actual application.

Indeed, temp string lives only during function call. What happened here is that in some circustances when mixing String and const char * parameters, const char * gets converted to String temporary, then back to const char *, then temporary is detroyed and dangling const char * returned.

Quote:

Return type is a value, not a reference. So, no temporaries ...

And that is exactly the problem. That return value is temporary one level up and gets converted to const char *....

It is very tricky indeed. Actually the version posted here was not final, it needed more fixes for other situations (namely enums). Hopefully trunk version is now ok. Full test is in "autotest/decode". Also the error only appears with MSC, but I have checked, the problem is not in the compiler.

[Updated on: Fri, 03 July 2020 10:10]

Report message to a moderator

Send a private message to this user

[Message index]

		get_i By: mirek on Sun, 14 June 2020 19:10
		Re: get_i By: Novo on Tue, 16 June 2020 17:45
		Re: get_i By: mirek on Tue, 16 June 2020 17:52
		Re: get_i By: Novo on Tue, 16 June 2020 18:02
		Re: get_i By: Novo on Tue, 16 June 2020 18:21
		Re: get_i By: mirek on Tue, 16 June 2020 21:20
		Re: get_i By: Novo on Tue, 16 June 2020 22:15
		Re: get_i By: mirek on Wed, 17 June 2020 00:01
		Re: get_i By: Novo on Wed, 17 June 2020 07:01
		Re: get_i By: Novo on Wed, 17 June 2020 07:09
		Re: get_i By: Novo on Wed, 17 June 2020 07:23
		Re: get_i By: Novo on Wed, 17 June 2020 07:50
		Re: get_i By: mirek on Wed, 17 June 2020 09:35
		Re: get_i By: mirek on Wed, 17 June 2020 13:03
		Re: get_i By: Novo on Wed, 17 June 2020 19:00
		Re: get_i By: Novo on Wed, 17 June 2020 19:14
		Re: get_i By: mirek on Wed, 17 June 2020 21:03
		Re: get_i By: Novo on Wed, 17 June 2020 23:21
		Re: get_i By: Novo on Wed, 17 June 2020 23:59
		Re: get_i By: mirek on Thu, 18 June 2020 08:39
		Re: get_i By: Novo on Thu, 18 June 2020 23:56
		Re: get_i By: Novo on Tue, 16 June 2020 21:22
		Re: get_i By: mirek on Tue, 16 June 2020 21:25
		Re: get_i By: Novo on Tue, 16 June 2020 21:42
		Re: get_i By: Novo on Wed, 17 June 2020 06:39
		Re: get_i By: mirek on Mon, 29 June 2020 19:23
		Re: get_i By: Novo on Thu, 02 July 2020 22:08
		Re: get_i By: mirek on Fri, 03 July 2020 09:38
		Re: get_i By: Novo on Fri, 03 July 2020 18:53
		Re: get_i By: mirek on Fri, 03 July 2020 19:03

Previous Topic:	ide: Assist parsing now parallel and some of it background
Next Topic:	2020.1 officially released

Goto Forum:

-=] Back to Top [=-

[ Syndicate this forum (XML) ] [

] [

PDF

]

Current Time: Wed Apr 30 01:07:51 CEST 2025

Total time taken to generate the page: 0.04323 seconds