Re: Branch review request

2013-08-09 Thread Sebastian Kügler
On Friday, August 09, 2013 15:49:33 Ivan Čukić wrote: > > It doesn't belong into plasma-framework, > > The original mail *was* about plasma-framework - the thread was > started when Kevin removed the add_definitions("-std=c++0x") from the > repo's top CMakeLists.txt > http://mail.kde.org/pipermail

Re: Branch review request

2013-08-09 Thread Ivan Čukić
p.s. I'm a bit of a safe-coding buff lately - if something removes potential "stupid bugs" and doesn't introduce a performance overhead - I'm all into it (like for vs foreach and similar). ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kd

Re: Branch review request

2013-08-09 Thread Ivan Čukić
> It doesn't belong into plasma-framework, The original mail *was* about plasma-framework - the thread was started when Kevin removed the add_definitions("-std=c++0x") from the repo's top CMakeLists.txt http://mail.kde.org/pipermail/plasma-devel/2013-June/025747.html > and you didn't answer my "w

Re: Branch review request

2013-08-09 Thread Sebastian Kügler
On Friday, August 09, 2013 15:30:08 Ivan Čukić wrote: > > - A custom d-pointer implementation? This seems like overkill and > > To quote Aaron: > "would love to see all the dptrs move to your spiffy d_ptr smart > pointer as well." > after this blog post of mine: > http://ivan.fomentgroup.org/blog/

Re: Branch review request

2013-08-09 Thread Ivan Čukić
> - A custom d-pointer implementation? This seems like overkill and To quote Aaron: "would love to see all the dptrs move to your spiffy d_ptr smart pointer as well." after this blog post of mine: http://ivan.fomentgroup.org/blog/2013/06/22/d-ptr-the-modern-way/ >> *(2) The other thing I wanted t

Re: Branch review request

2013-08-09 Thread Sebastian Kügler
On Saturday, August 03, 2013 19:34:32 Ivan Čukić wrote: > I've realized that it will be painful if I wait until everything is > finished to merge into master, and that there is not much reason for it to > be so since the master is in a playground mode anyway. > > One of the reasons for the reques

Re: Branch review request

2013-08-03 Thread Ivan Čukić
A small addendum - the ivan/shell-switching branch is sufficient - no need for the scratch repos. The loaders are quite simplified. The blank shell behaves like failsafe - it loads xterm application when shown - don't know how it behaves when xterm is not installed. -- Cheerio, Ivan -- While

Branch review request

2013-08-03 Thread Ivan Čukić
Hi all, I've realized that it will be painful if I wait until everything is finished to merge into master, and that there is not much reason for it to be so since the master is in a playground mode anyway. One of the reasons for the request is that I'd like to do a few things in parallel to th