Page MenuHomePhabricator

walac (Wander Lairson Costa)
User

Projects

User does not belong to any projects.

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Saturday

  • Clear sailing ahead.

User Details

User Since
Jun 8 2020, 6:04 AM (4 w, 3 d)
Availability
Available

Recent Activity

Tue, Jul 7

walac updated the diff for D12037: Implement eina_thread for native windows.

Fix minor compatibility corner cases with the posix counterpart.

Tue, Jul 7, 11:12 AM · efl

Mon, Jul 6

walac requested review of D12043: eo_test_general.c: Make eo_signals tests pass on Windows.
Mon, Jul 6, 1:17 PM · efl

Fri, Jul 3

walac closed D12041: Protect EINA_(UN)LIKELY with parenthesis around the expr.
Fri, Jul 3, 9:18 AM · efl
walac updated subscribers of D12041: Protect EINA_(UN)LIKELY with parenthesis around the expr.

Patch does not apply ...

Fri, Jul 3, 7:07 AM · efl
walac updated the diff for D12041: Protect EINA_(UN)LIKELY with parenthesis around the expr.

Fix patch to apply to HEAD

Fri, Jul 3, 7:06 AM · efl

Thu, Jul 2

walac requested review of D12041: Protect EINA_(UN)LIKELY with parenthesis around the expr.
Thu, Jul 2, 2:39 PM · efl

Tue, Jun 30

walac updated the diff for D12037: Implement eina_thread for native windows.

Add tests and fix bugs

Tue, Jun 30, 11:39 AM · efl
walac added inline comments to D12037: Implement eina_thread for native windows.
Tue, Jun 30, 6:49 AM · efl
walac updated the diff for D12037: Implement eina_thread for native windows.

Address raster's review comments

Tue, Jun 30, 6:45 AM · efl

Mon, Jun 29

walac updated the diff for D12037: Implement eina_thread for native windows.

Fix undefined behavior in eina_thread_self

Mon, Jun 29, 1:17 PM · efl
walac updated the diff for D12037: Implement eina_thread for native windows.

Use native windows threads in mingw

Mon, Jun 29, 12:53 PM · efl
walac added inline comments to D12037: Implement eina_thread for native windows.
Mon, Jun 29, 10:08 AM · efl
walac added inline comments to D12037: Implement eina_thread for native windows.
Mon, Jun 29, 10:04 AM · efl
walac requested review of D12037: Implement eina_thread for native windows.
Mon, Jun 29, 7:50 AM · efl

Fri, Jun 26

walac abandoned D12024: windows: Fix file mapping.
Fri, Jun 26, 7:31 AM · efl
walac accepted D12031: Windows: fix eina_file_map_new().
Fri, Jun 26, 6:32 AM · efl
walac accepted D12033: Native Windows: Eina: Resolve bad comparison while using windows strerror_s.
Fri, Jun 26, 6:20 AM · efl

Thu, Jun 25

walac added inline comments to D12031: Windows: fix eina_file_map_new().
Thu, Jun 25, 6:57 AM · efl

Wed, Jun 24

walac updated the diff for D12021: Fix eina file thread test on Windows.

Address review comments

Wed, Jun 24, 6:32 AM · efl
walac added inline comments to D12021: Fix eina file thread test on Windows.
Wed, Jun 24, 6:16 AM · efl
walac updated the diff for D12024: windows: Fix file mapping.

Fix struct layout compatibility

Wed, Jun 24, 6:07 AM · efl

Tue, Jun 23

walac requested review of D12024: windows: Fix file mapping.
Tue, Jun 23, 1:23 PM · efl
walac requested review of D12023: eina_test_lock.c: Implement clock_gettime for Windows.
Tue, Jun 23, 1:20 PM · efl
walac added a comment to D12022: windows: Fix path for file sanitization.

i've looked at https://docs.microsoft.com/en-us/windows/win32/fileio/naming-a-file#fully-qualified-vs-relative-paths and i indeed see it can begin with \ (but not /) :

"A file name is relative to the current directory if it does not begin with one of the following etc... "
"A single backslash, for example, "\directory" or "\file.txt". This is also referred to as an absolute path."

i don't know if you should also consider /

Tue, Jun 23, 9:42 AM · efl
walac requested review of D12022: windows: Fix path for file sanitization.
Tue, Jun 23, 8:21 AM · efl
walac updated the diff for D12021: Fix eina file thread test on Windows.

Update the patch based on reviewer feedback

Tue, Jun 23, 8:16 AM · efl
walac updated the diff for D12021: Fix eina file thread test on Windows.

Fix typo

Tue, Jun 23, 7:41 AM · efl
walac added inline comments to D12021: Fix eina file thread test on Windows.
Tue, Jun 23, 7:37 AM · efl
walac updated the diff for D12021: Fix eina file thread test on Windows.

Fix patch update

Tue, Jun 23, 7:36 AM · efl
walac updated the diff for D12021: Fix eina file thread test on Windows.

Fix negative path length calculation

Tue, Jun 23, 7:35 AM · efl
walac requested review of D12021: Fix eina file thread test on Windows.
Tue, Jun 23, 6:55 AM · efl

Mon, Jun 22

walac added a comment to D11993: eina: Change EAPI to EINA_API in Eina Library.

@walac your prefer this patch which modifies 143 files, ONLY for eina, to my patch, which modifies 5 files for 4 libraries, with the same result ?? that's crazy

Mon, Jun 22, 7:26 AM · efl

Fri, Jun 19

walac added a comment to D11993: eina: Change EAPI to EINA_API in Eina Library.

do you realize that, with this kind of changes, you are going to change almost all the files of the EFL ?

Fri, Jun 19, 7:39 AM · efl
walac accepted D11993: eina: Change EAPI to EINA_API in Eina Library.

That patch seems a step towards in the right direction IMHO. Things get tricky when EFL try to use each other since, in some places, EAPI must be __declspec(dllimport) and in others __declspec(dllexport), which leads walking on thin ice by playing around with the preprocessor. I understand that each library should define its XXX_API macro. The static library support is a win, but not the main point of the patch.

Fri, Jun 19, 5:31 AM · efl

Wed, Jun 17

walac accepted D11983: Fix ENOMEM message on Windows and solaris.
Wed, Jun 17, 5:07 AM · efl

Mon, Jun 15

walac added a comment to D11970: efl_check.h: Replace stack allocated array by heap allocated.

Gettign of a tons of errors with this patch and cxx bindings enabled. One example (there are more):

[219/398] Compiling C++ object 'src/tests/eina_cxx/6b3491e@@eina_cxx_suite@exe/eina_cxx_suite.cc.o'
In file included from ../src/tests/eina_cxx/eina_cxx_suite.h:8,
                 from ../src/tests/eina_cxx/eina_cxx_suite.cc:20:
../src/tests/eina_cxx/../efl_check.h: In function ‘int _efl_test_option_disp(int, char**, const Efl_Test_Case*)’:
../src/tests/eina_cxx/../efl_check.h:137:23: warning: ISO C++ forbids converting a string constant to ‘char*’ [-Wwrite-strings]
  137 |              nav[0] = "valgrind";
      |                       ^~~~~~~~~~
../src/tests/eina_cxx/../efl_check.h:138:23: warning: ISO C++ forbids converting a string constant to ‘char*’ [-Wwrite-strings]
  138 |              nav[1] = "--trace-children=yes";
Mon, Jun 15, 5:30 AM · efl
walac updated the diff for D11970: efl_check.h: Replace stack allocated array by heap allocated.

Make C++ compiler happy

Mon, Jun 15, 5:27 AM · efl

Fri, Jun 12

walac added a comment to D11970: efl_check.h: Replace stack allocated array by heap allocated.

@vtorri done

Fri, Jun 12, 1:54 PM · efl
walac updated the diff for D11970: efl_check.h: Replace stack allocated array by heap allocated.

Replace malloc by alloca

Fri, Jun 12, 1:53 PM · efl
walac added inline comments to D11971: eina_test_file.c: Do not test errno if the function doesn't fail.
Fri, Jun 12, 1:29 PM · efl
walac updated the diff for D11971: eina_test_file.c: Do not test errno if the function doesn't fail.

Remove all references to errno in eina_test_file.c

Fri, Jun 12, 1:27 PM · efl
walac requested review of D11971: eina_test_file.c: Do not test errno if the function doesn't fail.
Fri, Jun 12, 11:51 AM · efl
walac requested review of D11970: efl_check.h: Replace stack allocated array by heap allocated.
Fri, Jun 12, 11:40 AM · efl