Re: Review Request 116097: No longer use strlcpy in kstartupconfig

2014-03-15 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/116097/#review52983 --- This review has been submitted with commit

Re: Review Request 116097: No longer use strlcpy in kstartupconfig

2014-03-15 Thread Alexander Richardson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/116097/ --- (Updated March 15, 2014, 3:40 p.m.) Status -- This change has been

Re: Review Request 116097: No longer use strlcpy in kstartupconfig

2014-03-14 Thread Jaime Torres Amate
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/116097/#review52934 --- It works for me (I get kstartupconfig5 built). If it works in

Re: Review Request 116097: No longer use strlcpy in kstartupconfig

2014-03-04 Thread Alexander Richardson
On Feb. 26, 2014, 9:32 p.m., Rolf Eike Beer wrote: When you start poking at this code, why not kill the entire fopen/fgets/fixed size buffer stuff and replace it with std streams or something like that? I don't really know what kstartupconfig is supposed to do. This means I don't feel

Review Request 116097: No longer use strlcpy in kstartupconfig

2014-02-26 Thread Alexander Richardson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/116097/ --- Review request for kde-workspace. Repository: kde-workspace

Re: Review Request 116097: No longer use strlcpy in kstartupconfig

2014-02-26 Thread Alexander Richardson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/116097/ --- (Updated Feb. 26, 2014, 7:11 p.m.) Review request for kde-workspace.

Re: Review Request 116097: No longer use strlcpy in kstartupconfig

2014-02-26 Thread Rolf Eike Beer
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/116097/#review50982 --- When you start poking at this code, why not kill the entire