D13079: use a consistent style to handle fork()

2018-05-23 Thread Albert Astals Cid
aacid added inline comments. INLINE COMMENTS > pam_kwallet.c:441 > +switch (pid) { > +case 0: > +break; This construct makes the code harder to read imho, previously it was clear that the parent (i.e. pid != 0) was being exited and the child (outside the if) left to live, now y

D13079: use a consistent style to handle fork()

2018-05-23 Thread Rolf Eike Beer
dakon created this revision. dakon added a reviewer: aacid. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. dakon requested review of this revision. REPOSITORY R107 KWallet PAM Integration REVISION DETAIL https://phabricator.kde.org/D13