[Bug 444605] New: kdebluetooth4-0.2-12.17: reference to local variable 'val ' returned
https://bugzilla.novell.com/show_bug.cgi?id=444605 Summary: kdebluetooth4-0.2-12.17: reference to local variable 'val' returned Product: openSUSE 11.1 Version: Factory Platform: All OS/Version: openSUSE 11.0 Status: NEW Severity: Normal Priority: P5 - None Component: Other AssignedTo: bnc-team-screening@forge.provo.novell.com ReportedBy: dcb314@hotmail.com QAContact: qa@suse.de Found By: --- I just compiled the Suse Linux factory package kdebluetooth4-0.2-12.17 and I noticed the following /usr/src/packages/BUILD/kdebluetooth4-0.2/src/obexsession/obexsession.cpp:142: warning: reference to local variable 'val' returned I checked the source code const QString& ObexSession::getCurrentPath() { QString val; QDBusReply<QString> reply = session->call("GetCurrentPath"); if (reply.isValid()) { val = reply.value(); return val; and I agree with the compiler. Suggest code re-work. Return by value might be a good idea. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=444605 Cyril Hrubis <chrubis@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|bnc-team-screening@forge.provo.novell.com |tpatzig@novell.com -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=444605 User tpatzig@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=444605#c1 Tom Patzig <tpatzig@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED Resolution| |FIXED --- Comment #1 from Tom Patzig <tpatzig@novell.com> 2009-01-18 17:16:53 MST --- Thanks for reporting. Today i released kdebluetooth4-0.3. It now provides full bluez4 support. I fixed these and a lot of other warnings within this 0.3 release. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=444605 Cyril Hrubis <chrubis@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|bnc-team-screening@forge.provo.novell.com |tpatzig@novell.com -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=444605 User tpatzig@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=444605#c1 Tom Patzig <tpatzig@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED Resolution| |FIXED --- Comment #1 from Tom Patzig <tpatzig@novell.com> 2009-01-18 17:16:53 MST --- Thanks for reporting. Today i released kdebluetooth4-0.3. It now provides full bluez4 support. I fixed these and a lot of other warnings within this 0.3 release. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
participants (1)
-
bugzilla_noreply＠novell.com