Page MenuHomePhabricator

raster (Carsten Haitzler (Rasterman))Administrator
Keeper of the rabid stoat

Projects (8)

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Thursday

  • Clear sailing ahead.

User Details

User Since
Jan 27 2013, 4:24 PM (329 w, 1 d)
Roles
Administrator
Availability
Available

Behold... the rabid stoat! Prostrate yourselves before it before it is too late!

Recent Activity

Fri, May 17

raster closed D8909: Eet: remove useless inclusion of Evil.h.
Fri, May 17, 3:54 AM · efl
raster closed D8908: bin/efl and ecore: include evil_private.h when appropriate.
Fri, May 17, 3:53 AM · efl
raster accepted D8908: bin/efl and ecore: include evil_private.h when appropriate.
Fri, May 17, 3:53 AM · efl
raster accepted D8909: Eet: remove useless inclusion of Evil.h.
Fri, May 17, 3:53 AM · efl

Thu, May 16

raster updated the task description for T6726: E/EFL Flat Theme.
Thu, May 16, 9:30 AM · efl: widgets, Restricted Project, enlightenment-git, Restricted Project
raster updated the task description for T6726: E/EFL Flat Theme.
Thu, May 16, 9:10 AM · efl: widgets, Restricted Project, enlightenment-git, Restricted Project
raster updated the task description for T6726: E/EFL Flat Theme.
Thu, May 16, 9:08 AM · efl: widgets, Restricted Project, enlightenment-git, Restricted Project
raster closed D7602: screensaver: Allow to set hibernate on blank into blanking setup.
Thu, May 16, 5:54 AM · enlightenment-git
raster accepted D7602: screensaver: Allow to set hibernate on blank into blanking setup.

:) :) :)

Thu, May 16, 5:52 AM · enlightenment-git
raster requested changes to D7639: emixer: Add volume channels control on emixer.

hmmm... we're in massive conflict land here. :(

Thu, May 16, 5:49 AM · enlightenment-git

Wed, May 15

raster closed D8893: Evil: move the inclusion of all headers in evil_private.h.
Wed, May 15, 4:35 AM · efl
raster accepted D8893: Evil: move the inclusion of all headers in evil_private.h.
Wed, May 15, 4:26 AM · efl

Tue, May 14

raster requested changes to D8891: Evil: remove usage of EVIL_UNUSED and just cast to (void).

i'm not sure i like this change - the UNUSED macro is much nicer/better... :|

Tue, May 14, 2:45 AM · efl
raster added a comment to T7958: don't use long in eina value tests as they are 32 bits types on Windows.

Well they were already longs - I just made it explicit with the cast. The test is broken on 32bit already. In fact eina_value is broken on 32bit SPECIFICALLY with longs and you are right - windows on 64bit here will behave like 32bit in these cases. Already broken. :(

Tue, May 14, 2:18 AM · efl

Mon, May 13

raster added a comment to D8095: Add json-c dependency.

oh while i'm at it...

Mon, May 13, 7:51 AM
raster requested changes to D8095: Add json-c dependency.

i had some stale forecasts config that didn't have a server to contact in it....

Mon, May 13, 7:44 AM

Fri, May 10

raster updated the task description for T6726: E/EFL Flat Theme.
Fri, May 10, 3:58 AM · efl: widgets, Restricted Project, enlightenment-git, Restricted Project
raster updated the task description for T6726: E/EFL Flat Theme.
Fri, May 10, 3:20 AM · efl: widgets, Restricted Project, enlightenment-git, Restricted Project
raster updated the task description for T6726: E/EFL Flat Theme.
Fri, May 10, 3:19 AM · efl: widgets, Restricted Project, enlightenment-git, Restricted Project

Thu, May 9

raster updated the task description for T6726: E/EFL Flat Theme.
Thu, May 9, 10:23 AM · efl: widgets, Restricted Project, enlightenment-git, Restricted Project
raster updated the task description for T6726: E/EFL Flat Theme.
Thu, May 9, 10:13 AM · efl: widgets, Restricted Project, enlightenment-git, Restricted Project
raster updated the task description for T6726: E/EFL Flat Theme.
Thu, May 9, 10:07 AM · efl: widgets, Restricted Project, enlightenment-git, Restricted Project
raster added a comment to D8106: canvas map: introduce a new texture mapping for better quality..

hermet. you need to take a good hard loo at this. i''m seeing coordinates that are negative and i think other out of bounds coordinates as well. you need to have extensive tests with different resolution input, output, many angles and so on... i've already put a quick and dirty fixup for negative uv coords, but even with that i see rendering bugs in some cases. :)

Thu, May 9, 10:01 AM · efl

Wed, May 8

raster reopened D8854: Evil: remove localtime_r as mingw-w64 defines it when _POSIX_C_SOURCE is defined.

it seems this breaks on osx:

Wed, May 8, 6:00 AM · efl
raster closed D8854: Evil: remove localtime_r as mingw-w64 defines it when _POSIX_C_SOURCE is defined.
Wed, May 8, 4:47 AM · efl
raster accepted D8854: Evil: remove localtime_r as mingw-w64 defines it when _POSIX_C_SOURCE is defined.
Wed, May 8, 4:47 AM · efl

Tue, May 7

raster closed D8842: eina_file: speed up listing of directories on Windows.
Tue, May 7, 3:16 AM · efl
raster accepted D8842: eina_file: speed up listing of directories on Windows.
Tue, May 7, 3:15 AM · efl
raster added a comment to T7907: about localtime_r (unix and windows).

so do you think that there is a problem if we define _POSIX_C_SOURCE just before including time.h in Elementary.h and Efl_UI.h ?

Tue, May 7, 2:55 AM · Restricted Project

Fri, May 3

raster closed D8814: replace strndup with eina_strndup, remove strndup definition in evil and elm_test_dnd.
Fri, May 3, 5:14 AM · efl
raster accepted D8814: replace strndup with eina_strndup, remove strndup definition in evil and elm_test_dnd.

much better! :)

Fri, May 3, 5:13 AM · efl
raster requested changes to D8814: replace strndup with eina_strndup, remove strndup definition in evil and elm_test_dnd.
Fri, May 3, 3:17 AM · efl

Thu, May 2

raster added a comment to D8792: eina_debug - fix init of cpu, bt and timer debug subsystems..

@bu5hm4n thanks for pointing that out :)

Thu, May 2, 9:10 AM · efl
raster added a comment to D8792: eina_debug - fix init of cpu, bt and timer debug subsystems..

move these just above the:

Thu, May 2, 9:09 AM · efl
raster added a comment to D8792: eina_debug - fix init of cpu, bt and timer debug subsystems..

how doe sit break bsd?

Thu, May 2, 9:00 AM · efl

Tue, Apr 30

raster closed D8785: eina_unicode: return NULL if allocation fails.
Tue, Apr 30, 1:25 AM · efl
raster accepted D8785: eina_unicode: return NULL if allocation fails.
Tue, Apr 30, 1:23 AM · efl

Fri, Apr 26

raster updated the task description for T6726: E/EFL Flat Theme.
Fri, Apr 26, 5:46 AM · efl: widgets, Restricted Project, enlightenment-git, Restricted Project
raster added a comment to D8678: eo: add events to track the ownership status of an Eo object.

expedite is not the right test for this. things like elementary - like genlist and things that create, destroy and move objects in and out of parents and have complex call trees - those are the things u need to look at

Fri, Apr 26, 2:29 AM · efl

Wed, Apr 24

raster added a comment to D8678: eo: add events to track the ownership status of an Eo object.

what they said. this is going to be costly...

Wed, Apr 24, 2:10 PM · efl

Tue, Apr 23

raster added a comment to T7821: terminology double char width computation breaks mutt.

hmm whilst unicode says they are emoji.. they don't come out as ones according to terminal apps so yeah - keep the entire range out.

Tue, Apr 23, 3:19 AM · Terminology
raster added a comment to T7821: terminology double char width computation breaks mutt.

the problem is 25AA, 25AB, 25B6, 25C0, 25FB->25FE are emoji...

Tue, Apr 23, 1:45 AM · Terminology

Apr 18 2019

raster added a comment to D8572: elm_entry: speed up setting text (check if same text is already set), fix setting same text pointer.

oh i had an idea over lunch... store sd->text in a local var and do the free at the end of the function. then it can't ever be an invalid pointer. the check for it being the same ptr can avoid doing the duplicate and the free to as it does now so store and move the free before the done label and there will be no crash. right?

Apr 18 2019, 6:16 AM · efl
raster requested changes to D8572: elm_entry: speed up setting text (check if same text is already set), fix setting same text pointer.

small change i think ... it's less code and less prone to error.

Apr 18 2019, 5:09 AM · efl
raster added a comment to D8578: Freeing Global Memory list on destructor.

but those functions then access the globals anyway so it's just pushing the problem away one level into another function, and if someone forgets a RemoveSharedStyle() call just like the list remove above - same thing happens. it's not really any different. these functions are indeed good to avoid copy & paste code if you have lots of places that do the same list remove and to be fair - efl has a lot of things like this that have crept in over many years.this is an area that efl could do with some attention to look for "copy & paste" code even if it looks different on the surface and possibly has minor difference (that could be bugs - careful thought and attention would need to be applied to see if they are really intended or not). if these bits of code can be refactored to share more common functions like you describe above where there is duplication, code size can be reduced and the chances of future problems where a list remove is forgotten in 1 case but not the other reduce. so to some extent you have a point, but in this case there isn't anything really to gain. indirection to a function which is all of 1 or 2 lines of code is generally not that useful. if its a larger blob of complexity then it'd probably worth a look. :) i hope this serves as some encouragement and guidance. in this case you found a bug, but your solution is IMHO with the above functions won't help IMHO, but there are enough other cases where this is really needed and you'll get little argument over :) i know the code exists. i know i've been guilty of doing it myself often enough. :)

Apr 18 2019, 5:01 AM · efl
raster added a comment to D8578: Freeing Global Memory list on destructor.

@bu5hm4n is right - this has nothing to do with globals. it'd be the same as if obj->styles = eina_list_remove(obj->styles, ts); was missing in the free func for the obj.

Apr 18 2019, 1:37 AM · efl

Apr 17 2019

raster closed D8633: eina_vpath: port to Windows.
Apr 17 2019, 9:44 AM · efl
raster accepted D8633: eina_vpath: port to Windows.

;]

Apr 17 2019, 7:30 AM · efl
raster accepted D8632: efl-net.pc.in : honor datarootdir.

;]

Apr 17 2019, 7:29 AM · efl
raster accepted D8552: evas gl: move to floating point coordinate system..

yup. no big difference. ok. i'm comfortable with moving to float entirely.

Apr 17 2019, 1:29 AM · efl

Apr 16 2019

raster added a comment to T7788: Memory Leak efl_add.

there isn't a single control. evas has various caches, edje has, there are mempools and those are set up by code, and so on etc. etc. - so you'd have to tweak various things to force free and some cant be force freed at all without modifying the code to no longer use a pool etc.

Apr 16 2019, 3:22 PM · enlightenment-git
raster added a comment to T7788: Memory Leak efl_add.

you'd have to stress it and re-use a lot of objects to see if there is a leak. we do have memory pools and trash etc. like @bu5hm4n says - we alloc and keep around that memory then re-use it again. you'd have to do a lot of allocs over and over and then see the mem footprint keep going up and up and never stabilizing. i adapted your code a bit to do this with a timer ever 0.01 sec:

Apr 16 2019, 11:09 AM · enlightenment-git
raster closed D8621: evil: remove tmp and home dir get.
Apr 16 2019, 10:00 AM · efl
raster accepted D8621: evil: remove tmp and home dir get.

;]

Apr 16 2019, 9:58 AM · efl

Apr 15 2019

raster closed D8602: Windows : fix the definition of EAPI.
Apr 15 2019, 5:34 AM · efl
raster accepted D8602: Windows : fix the definition of EAPI.

;]

Apr 15 2019, 5:32 AM · efl
raster added a comment to T7788: Memory Leak efl_add.

because the canvas didn't go through a render cycle - nothing changed visually thus it didn't render. it has to actually go through a full cycle. try show some of those rects and leave them there for 1 frame.... then delete them.

Apr 15 2019, 12:53 AM · enlightenment-git

Apr 14 2019

raster closed T7788: Memory Leak efl_add as Invalid.

because evas object deletions are DEFERRED. they require render cycles to drive the deletion. they will be kept around for 2 render cycles so evas can do a prev state vs current state check., imagine u delete a visible object - how can evas know what changes - what area is occupied that now needs re-rendering? was it visible or not, was it set to a color of 0 0 0 0 which EFFECTIVELY made it invisible to rendering even if it was shown... thus objects invisibly have extra references to them that are removed if delete flags are set by the render cycles, so the above case is just "invalid" in that any real process will never do the above and will have a fairly limited upper bound on how many objects will be created then deleted before a render cycle happens. if there really is a very big spate of creating invisible objects (e.g. you're using evas as an image processing engine where you might load an image file into an image object then save it out as another format) then there is evas_norender(evas); which does a render cycle without doing the actual rendering thus driving the object deletion pump. it's an incredibly rare day this is ever needed and if it is used, any updates from objects that really did need a render will be lost. you would need to manually add a damaged region for the whole canvas output to force a full re-render to avoid this if this ever were the case.

Apr 14 2019, 4:01 AM · enlightenment-git

Apr 11 2019

raster closed D8598: elementary: rename ELEMENTARY_BUILD to EFL_BUILD like all other libs.
Apr 11 2019, 9:59 AM · efl
raster accepted D8598: elementary: rename ELEMENTARY_BUILD to EFL_BUILD like all other libs.
Apr 11 2019, 9:58 AM · efl

Apr 9 2019

raster closed D8583: eldbus unit test: fix warnings.
Apr 9 2019, 4:37 AM · Restricted Project
raster accepted D8583: eldbus unit test: fix warnings.

:)

Apr 9 2019, 4:36 AM · Restricted Project

Apr 7 2019

raster added a commit to T7728: eina_test_vpath.c : getuid() does not exists on Windows: rEFL954a534bc0e7: remove vpath test for user dir the test was broken and fixing is insane.
Apr 7 2019, 5:19 AM · Restricted Project
raster closed T7728: eina_test_vpath.c : getuid() does not exists on Windows as Resolved by committing rEFLe25ba74d2ba4: remove vpath test for user dir the test was broken and fixing is insane.
Apr 7 2019, 5:19 AM · Restricted Project
raster added a comment to T7728: eina_test_vpath.c : getuid() does not exists on Windows.

actually struct passwd and getpwuid are probably a major problem...

Apr 7 2019, 5:02 AM · Restricted Project

Apr 5 2019

raster added a comment to D8552: evas gl: move to floating point coordinate system..

I mean weak performance is "weak point" as you can see the video attached.

Apr 5 2019, 3:46 AM · efl

Apr 4 2019

raster added a comment to D8552: evas gl: move to floating point coordinate system..

But obviously, efl animation show us very weak performance than other decent frameworks.

Apr 4 2019, 8:58 AM · efl
raster requested changes to D8552: evas gl: move to floating point coordinate system..

this has a downside. it doubles the memory needed for coord storage/copying to gpu etc. .... that will mean some performance drop and the only benefits are when using msaa... but even then evas object geometry is done with integers only so it can only help with map and even then map reduces to integers anyway at the engine api....

Apr 4 2019, 1:15 AM · efl
raster accepted D8551: efreetd: remove a tentative global variable definition.

that is indeed a bug... thanks!

Apr 4 2019, 1:11 AM · efl

Apr 3 2019

raster requested changes to D8540: elementary: add a queue to postpone object destruction when necessary..

this does have a problem - nothing ever calls eina_freeq_reduce()/eina_freeq_clear()/eina_freeq_reduce(), so anything put on this freeq will never get freed until process exit.

Apr 3 2019, 4:22 PM · efl

Apr 2 2019

raster closed T7685: Video wallpapers for e broken - doesn't seem to propagate update rects as Resolved by committing rEFL07f0044d281b: evas image - fix proxy change propagation to fix e video wallpapers.
Apr 2 2019, 10:25 AM · efl
raster closed T7776: compilation error on Windows : elm_naviframe_item_push() is redeclared differently as Resolved by committing rEFL96f1e2970921: elm naviframe - remove duplicated func prototype in headers.
Apr 2 2019, 5:59 AM · Restricted Project

Apr 1 2019

raster added a comment to D8478: evas_render: Process deferred callback in the sync render case..

@Hermet - actually gl rendering is still sync mode... :)

Apr 1 2019, 7:10 AM · efl

Mar 30 2019

raster closed T6247: Please install engine headers again as Wontfix.

:)

Mar 30 2019, 6:56 AM · efl

Mar 28 2019

raster closed T7764: evasgl usage broken as Resolved by committing rEFL3439134ea1f8: Revert "evas gl engines - avoid getting context if possible".
Mar 28 2019, 5:19 PM · regression, efl (efl-1.22), efl: rendering

Mar 19 2019

raster accepted D8106: canvas map: introduce a new texture mapping for better quality..

aaah ok. that's a good selection there. thanks! :) so impact can vary from almost nothing to "pretty big". at least it's an option - aa and smooth on at the same time i think is good enough. if it was not that much more i was going to suggest we adopt it as the default "smooth". :)

Mar 19 2019, 4:06 PM · efl
raster raised the priority of T7685: Video wallpapers for e broken - doesn't seem to propagate update rects from High to Showstopper Issues.
Mar 19 2019, 4:03 PM · efl

Mar 15 2019

raster added a comment to T7360: Evas/Edje animations not in sync .

I'm not going to have time for a few weeks...

Mar 15 2019, 2:54 AM · efl: layout engine, efl: rendering, efl (efl-1.22), regression

Mar 12 2019

raster triaged T7737: efl-git 1.22.0beta1.61484 + enlightenment-git: icon menu properties not diplayed (wayland) as High priority.
Mar 12 2019, 10:39 PM · enlightenment-git, efl (efl-1.22)

Mar 10 2019

raster added a comment to T7734: windows : could not open src\lib\ecore\efl_loop_timer.eo.legacy.h.eo.legacy.h.

i can't reproduce... now that autotools is working enough again...

Mar 10 2019, 6:16 AM · efl (efl-1.23), Restricted Project
raster added a comment to T7734: windows : could not open src\lib\ecore\efl_loop_timer.eo.legacy.h.eo.legacy.h.

autotools was broken for me anyway. so i couldn't build with it. iut literally deleted files committed to git and failed to build part way thru

Mar 10 2019, 5:05 AM · efl (efl-1.23), Restricted Project

Mar 9 2019

raster abandoned D8099: evas: make convert_rgb_32 keep consistency.
Mar 9 2019, 7:22 AM · efl
raster commandeered D8099: evas: make convert_rgb_32 keep consistency.

going to close this as 4758f06e637239f981eedbaaf8c0d613b78e4417 solves it better by moving to tiled rendering

Mar 9 2019, 7:22 AM · efl
raster added a comment to D8099: evas: make convert_rgb_32 keep consistency.

on arm32 the tiled rotate is 2.4x faster vs the neon. the neon interestingly is also about 1.8x faster than the plain C code.

Mar 9 2019, 6:37 AM · efl
raster requested changes to D8099: evas: make convert_rgb_32 keep consistency.

ok i checked on arm32 and yeah - the neon is broken. but... it's half the speed of the tiled rotator... so why fix something that is slow when we can just switch to the faster one. :) i think its best to reject this and instead just remove any rotate code/paths that are not the tiled rotates... :)

Mar 9 2019, 5:49 AM · efl

Mar 8 2019

raster added a comment to D8095: Add json-c dependency.

forecasts doesn't work for me - only says it supports us zip codes (and i don't live there) and can't enter anything in the entry box there in settings anyway, so can't test it.

Mar 8 2019, 7:27 AM
raster added a comment to D8099: evas: make convert_rgb_32 keep consistency.

justr one thing i noted... why aren't we just doing tiled rotation by default anyway? in terms of pixels/sec its almost 2x the speed vs ye olde C. i'm not sure the neon would even be 2x the speed of C... so... anyway. i need to check more...

Mar 8 2019, 4:59 AM · efl
raster added a comment to D8099: evas: make convert_rgb_32 keep consistency.

argh... just realized this isn't used on aarch64 - only 32bit arm... i need more time to check on that as now my more accessible arms are all aarch64 :)

Mar 8 2019, 4:57 AM · efl

Mar 7 2019

raster added a comment to D8106: canvas map: introduce a new texture mapping for better quality..

what's the performance difference on a range of devices/architectures?

Mar 7 2019, 12:35 AM · efl

Mar 6 2019

raster added a comment to D8103: efl.file: use ENOENT as default error if file opening fails and errno isn't set.

do we need this if we have D8104 ?

Mar 6 2019, 2:17 AM · efl

Mar 5 2019

raster added a comment to D8099: evas: make convert_rgb_32 keep consistency.

give me a bit... just saw this and need to look at it more, BUT the if ((w & 1) || (h & 1)) is a simple "if width or height is odd (not a multiple of 2) in size). this is kind of lazy as the neon can still be used EXCEPT for the "edge pixels" where it can fall back to some C. I'll need to read this a bit carefully to figure out what's wrong (and maybe try some things out).

Mar 5 2019, 4:38 PM · efl

Mar 4 2019

raster closed T7097: something broken with gl when lots of windows open with x11 engine as Wontfix.

probably not even an error we can fix. this is probably something to bring up with the appropriate driver developers, but even then may never be fixed. also note you may hit a limit at 126/127/128 or so or maybe 254/255 as xorg has a limit on client connections and last time i hit it, it was 128 clients (e will have 1 connection at least itself to start). so you'll hit a wall/limit like that that will probably be far before you hit a ram limit these days. as i said the 20/30 limit smells like a driver/gpu limit in maximum number of contexts or something similar. we impose no limit in efl and even if we did it's be per process, not globally.

Mar 4 2019, 11:23 AM · efl: display system, efl: rendering

Mar 1 2019

raster added a comment to T7710: Impossible to build EFL on FreeBSD.

i couldn't say anymore - fbsd has since my build shot itself in the face. perhaps i did have some inotify lib installed... but i managed to build efl, e, rage and terminology...

Mar 1 2019, 4:34 PM · efl (efl-1.22)
raster added a comment to T7710: Impossible to build EFL on FreeBSD.

err sorry

Mar 1 2019, 10:51 AM · efl (efl-1.22)
raster added a comment to T7710: Impossible to build EFL on FreeBSD.

are you building with meson there? looks like it.

Mar 1 2019, 8:21 AM · efl (efl-1.22)
raster added a comment to T7710: Impossible to build EFL on FreeBSD.

bbdb2e5c23139d96b6fa57633f9a73e85ea6e7d6 i hope should fix this. i also found a few other loose ends regarding environ related things and fixed those too. my fbsd vm tho is having other issues (meson cant find libintl.h and it isn't respecting CFLAGS when looking for libintl.h ...:() so my fix above is a guess, but a decent one based on the fact that efl_core_proc_env.c was indeed missing the extern char **environ declaration and uour comment above that it works.

Mar 1 2019, 2:36 AM · efl (efl-1.22)

Feb 27 2019

raster added a comment to T7710: Impossible to build EFL on FreeBSD.

likely related to @bu5hm4n 's changes

Feb 27 2019, 12:30 AM · efl (efl-1.22)
raster assigned T7710: Impossible to build EFL on FreeBSD to bu5hm4n.
Feb 27 2019, 12:29 AM · efl (efl-1.22)

Feb 26 2019

raster added a comment to D7926: Eina: implement strtod in C locale and remove linkl against msvcr100..

see inline questions :)

Feb 26 2019, 2:37 PM · efl

Feb 24 2019

raster added a comment to T7707: termpty: compute exact line length in termpty_cellrow_get() breaks jed display.

hehehe yeah - tho .. xterm vs terminology should be enough to compare :)

Feb 24 2019, 12:47 PM · Terminology