Re: Review Request 130245: Fix Crash on JoyStick Config when js device did not contain regular joystick button

2017-09-20 Thread TOM Harrison via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/130245/ --- (Updated 九月 20, 2017, 11:55 p.m.) Status -- This change has been

Re: Review Request 130245: Fix Crash on JoyStick Config when js device did not contain regular joystick button

2017-09-20 Thread Albert Astals Cid
> On set. 16, 2017, 9:19 a.m., Albert Astals Cid wrote: > > Are you sure this fixes anything? > > > > Just before the lines you changed we have > > > > > > if ( ! buttonTbl->item(number, 0) ) > > buttonTbl->setItem(number, 0, new QTableWidgetItem()); > > > > Which will create the

Re: Review Request 130245: Fix Crash on JoyStick Config when js device did not contain regular joystick button

2017-09-20 Thread Albert Astals Cid
> On Sept. 16, 2017, 9:19 a.m., Albert Astals Cid wrote: > > Are you sure this fixes anything? > > > > Just before the lines you changed we have > > > > > > if ( ! buttonTbl->item(number, 0) ) > > buttonTbl->setItem(number, 0, new QTableWidgetItem()); > > > > Which will create