gawin (Filip Gawin)
User

Projects

User does not belong to any projects.

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Monday

  • Clear sailing ahead.

User Details

User Since
Nov 15 2017, 10:46 PM (92 w, 2 d)
Availability
Available

Recent Activity

Yesterday

gawin committed R40:850793397b92: Avoid copying if it's possible (authored by gawin).
Avoid copying if it's possible
Fri, Aug 23, 7:12 PM

Thu, Aug 22

gawin added a reviewer for D23351: Simplify ctors with default member initializer: Kate.
Thu, Aug 22, 3:42 PM · Kate
gawin added a comment to D23035: Simplify ctors with default member initializer.

I've tried to use arc for this. It created new PR. https://phabricator.kde.org/D23351

Thu, Aug 22, 3:41 PM · Kate
gawin updated subscribers of D23351: Simplify ctors with default member initializer.
Thu, Aug 22, 3:40 PM · Kate
gawin requested review of D23351: Simplify ctors with default member initializer.
Thu, Aug 22, 3:38 PM · Kate

Fri, Aug 9

gawin added a comment to D23035: Simplify ctors with default member initializer.

@cullmann: Filip Gawin, filip.gawin@zoho.com

Fri, Aug 9, 9:35 AM · Kate
gawin requested review of D23035: Simplify ctors with default member initializer.
Fri, Aug 9, 12:33 AM · Kate

Thu, Aug 8

gawin added a comment to D23010: Use range loop if possible.

It's ok, unless you don't "stand behind" this code. ;)

Thu, Aug 8, 12:11 PM · Kate
gawin updated the diff for D23010: Use range loop if possible.

Fixes to avoid detach (if possible).

Thu, Aug 8, 11:28 AM · Kate

Wed, Aug 7

gawin added a comment to D23010: Use range loop if possible.

qAsConst has problems if we want to take addr to object:

for (auto& pluginInfo : qAsConst(m_pluginList)) {
    m_name2Plugin[pluginInfo.saveName()] = &pluginInfo;
}

pluginInfo will be const ptr.

Wed, Aug 7, 9:11 PM · Kate
gawin added inline comments to D23010: Use range loop if possible.
Wed, Aug 7, 7:51 PM · Kate
gawin requested review of D23010: Use range loop if possible.
Wed, Aug 7, 7:46 PM · Kate

May 5 2019

gawin added a comment to D20737: Convert c arrays to std::array.

I'm not sure what has happened - I attached patch with 2 commits, but only one is visible.

May 5 2019, 9:00 PM · Dolphin
gawin updated the diff for D20737: Convert c arrays to std::array.

Dived code

May 5 2019, 8:59 PM · Dolphin

Apr 26 2019

gawin added a comment to D20737: Convert c arrays to std::array.

C++ 17 (only :( ) allows to change:

struct someClass{...};
Apr 26 2019, 4:01 PM · Dolphin

Apr 22 2019

gawin requested review of D20737: Convert c arrays to std::array.
Apr 22 2019, 1:55 PM · Dolphin