#33822 – missing return statement at end of non-void function

Bug 33822 - missing return statement at end of non-void function

Summary: missing return statement at end of non-void function

Status:	CLOSED FIXED

Alias:	None

Product:	Sisyphus
Classification:	Development
Component:	kf5-kwayland-devel (show other bugs)
Version:	unstable
Hardware:	e2k Linux

Importance:	P3 minor
Assignee:	Nobody's working on this, feel free to take it
QA Contact:	qa-sisyphus

URL:
Keywords:

Depends on:
Blocks:

Reported:	2017-08-29 09:55 MSK by Michael Shigorin
Modified:	2017-09-19 16:21 MSK (History)
CC List:	0 users

See Also:

Attachments
return -1; (258 bytes, patch) 2017-08-29 10:13 MSK, Michael Shigorin	no flags	Details \| Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description

Michael Shigorin 2017-08-29 09:55:12 MSK

kf5-kwayland-5.37.0-alt1.S1 не собирается при помощи lcc-1.21.20 на "Эльбрусе" из-за будто тривиальной ошибки (но интересно, был ли у компилятора шанс честно разобраться во всех возможных ветках в конечное время):

[ 81%] Building CXX object src/server/CMakeFiles/KF5WaylandServer.dir/textinput_interface_v2.cpp.o
cd /usr/src/RPM/BUILD/kwayland-5.37.0/BUILD/src/server && /usr/bin/c++   -DKF5WaylandServer_EXPORTS -DQT_CONCURRENT_LIB -DQT_CORE_LIB -DQT_GUI_LIB -DQT_NO_CAST_FROM_ASCII -DQT_NO_CAST_FROM_BYTEARRAY -DQT_NO_CAST_TO_ASCII -DQT_NO_DEBUG -DQT_NO_SIGNALS_SLOTS_KEYWORDS -DQT_NO_URL_CAST_FROM_STRING -DQT_USE_FAST_OPERATOR_PLUS -DQT_USE_QSTRINGBUILDER -D_GNU_SOURCE -D_LARGEFILE64_SOURCE -I/usr/src/RPM/BUILD/kwayland-5.37.0/BUILD/src/server -I/usr/src/RPM/BUILD/kwayland-5.37.0/src/server -I/usr/src/RPM/BUILD/kwayland-5.37.0/BUILD -isystem /usr/include/qt5 -isystem /usr/include/qt5/QtGui -isystem /usr/include/qt5/QtCore -isystem /usr/lib64/qt5/mkspecs/linux-g++ -isystem /usr/include/EGL -isystem /usr/include/qt5/QtConcurrent  -pipe -Wall -g -O3  -std=c++0x  -fno-exceptions -Wall -Wextra -Wcast-align -Wchar-subscripts -Wformat-security -Wno-long-long -Wpointer-arith -Wundef -Wnon-virtual-dtor -Woverloaded-virtual -Werror=return-type  -pedantic -O3 -DNDEBUG -fPIC -fvisibility=hidden -fvisibility-inlines-hidden   -fPIC -o CMakeFiles/KF5WaylandServer.dir/textinput_interface_v2.cpp.o -c /usr/src/RPM/BUILD/kwayland-5.37.0/src/server/textinput_interface_v2.cpp
lcc: "/usr/src/RPM/BUILD/kwayland-5.37.0/src/server/server_decoration_interface.cpp", line 120: error:
          missing return statement at end of non-void function
          "KWayland::Server::<unnamed>::modeWayland" [-Werror=return-type]
  }
  ^

1 error detected in the compilation of "/usr/src/RPM/BUILD/kwayland-5.37.0/src/server/server_decoration_interface.cpp".

Пока засунул туда return -1;.

Comment 1

Michael Shigorin 2017-08-29 10:13:30 MSK

Created attachment 7198 [details]
return -1;

PS: могу отправить NMU с приложенным патчиком и вдобавок

%ifarch e2k
%patch -p0
sed -i 's,-std=gnu90,-std=gnu9x,' CMakeLists.txt
%endif

для

lcc: ошибка: неправильный аргумент опции "-std=gnu90". Допустимые значения:
          c89, iso9899:1990, iso9899:199409, gnu89, c99, c9x, iso9899:1999,
          gnu99, gnu9x, c11, gnu11, c++98, gnu++98, c++03, gnu++03, c++0x,
          c++11, gnu++0x, gnu++11, c++14, gnu++14

Comment 2

Repository Robot 2017-09-18 12:00:45 MSK

kf5-kwayland-5.37.0-alt2.S1 -> sisyphus:

Mon Sep 18 2017 Sergey V Turchin <zerg@altlinux.org> 5.37.0-alt2.S1
- fix missing return statement (ALT#33822)

Comment 3

Michael Shigorin 2017-09-19 16:07:59 MSK

(В ответ на комментарий №2)
> kf5-kwayland-5.37.0-alt2.S1 -> sisyphus:
Спасибо!

(В ответ на комментарий №1)
> %ifarch e2k
> %patch -p0
> sed -i 's,-std=gnu90,-std=gnu9x,' CMakeLists.txt
> %endif
Здесь обождать, пока в lcc сделают? (повесил, взяли)

Comment 4

Sergey V Turchin 2017-09-19 16:21:13 MSK

(В ответ на комментарий №3)
> Здесь обождать, пока в lcc сделают? (повесил, взяли)
Ну, не всё ж костылями обвешивать...