[E-devel] Elementary Map Issues

2011-10-27 Thread Bluezery
Dear all, I am interested in elm_map widget. As far as I know, elm_map is a only widget use network resource in elementary. So, when the network condition is poor, some map images can not be visible while progress bar is still running. Due to above issues, I have checked the elm_map code

[E-devel] [Patch] elm_map: add image loading error handling

2011-10-30 Thread Bluezery
Dear all I made a bug fix for elm_map. When a downloaded image has an error, , elm_map removes a file and goes on just like download succeeded . However, it is not success, so it should be marked as FALSE for later retry. Can anyone review this simple patch? :) Index: elm_map.c

Re: [E-devel] [Patch] elm_map: add image loading error handling

2011-10-31 Thread Bluezery
happens when gi-file is NULL ? -Original Message- From: Bluezery [mailto:ohpowel@...] Sent: 31 October 2011 14:54 To: enlightenment-devel@... Subject: [E-devel] [Patch] elm_map: add image loading error handling Dear all I made a bug fix for elm_map. When a downloaded image has an error

Re: [E-devel] [elm_map] Fix up too many connections at the same time.

2011-11-01 Thread Bluezery
Pipeling is not always supported in server. Futuremore, I have already test pipelining but OSM (default elm_map server) might not support it. When capturing TCP packet, OSM uses http/1.0 spec (HTTP pipelining is supported when version is above 1.1) On Wed, Nov 2, 2011 at 3:18 AM, Joerg

[E-devel] [elm_map] ecore_file_download_full() should be used properly

2011-11-05 Thread Bluezery
Dear all, elm_map only checked that returned job is NULL. But ecore_file_download_full() returns EINA_FALSE when error occurred. So return value should be checked for proper usage. Please review this simple bug fix Index: src/lib/elm_map.c

[E-devel] [ecore_main] ecore_main_fd_handler_add() should return NULL when error occurs

2011-11-05 Thread Bluezery
Dear all, When ecore_main_fd_handler_add() failed to add poll on fd, it should return NULL. To prevent NULL Dereference. I made fdh NULL after free. Please review this patch. Index: src/lib/ecore/ecore_main.c === ---

[E-devel] [ecore_file] Check a return value of ecore_con_url_get()

2011-11-05 Thread Bluezery
Dear all _ecore_file_download_curl() does not check return value of even if ecore_con_url_get(), it can be failed. I add a simple error checking. Please review this patch. Index: src/lib/ecore_file/ecore_file_download.c === ---

Re: [E-devel] [elm_map] ecore_file_download_full() should be used properly

2011-11-06 Thread Bluezery
-job is not NULL? 2011. 11. 6. 오후 12:03에 Bluezery ohpo...@gmail.com님이 작성: Dear all, elm_map only checked that returned job is NULL. But ecore_file_download_full() returns EINA_FALSE when error occurred. So return value should be checked for proper usage. Please review this simple bug fix

[E-devel] [ecore_file] Add file remove for cleanup

2011-11-08 Thread Bluezery
Hi, I add ecore_file_remove() for removing a NULL file opened by fopen(dst, wb). Please review below. Index: src/lib/ecore_file/ecore_file_download.c === --- src/lib/ecore_file/ecore_file_download.c(리비전 64937) +++

[E-devel] [ecore_con] Bug fix when data is NULL.

2011-11-09 Thread Bluezery
Hi, If data is NULL, curl makes length field as -1 . length filed should be set by 0. Please review below. Index: src/lib/ecore_con/ecore_con_url.c === --- src/lib/ecore_con/ecore_con_url.c (리비전 64937) +++

Re: [E-devel] [ecore_con] Bug fix when data is NULL.

2011-11-09 Thread Bluezery
If it is http DELETE or GET, not POST, it can be 0. :-) On Thu, Nov 10, 2011 at 3:33 PM, Carsten Haitzler ras...@rasterman.com wrote: On Wed, 9 Nov 2011 18:29:49 +0900 Bluezery ohpo...@gmail.com said: Hi, If data is NULL, curl makes length field as -1 . length filed should be set by 0

[E-devel] [Patch][ecore_con] Refactoring curl port

2011-11-10 Thread Bluezery
Dear all, I have modified the way curl used in ecore_con_url.c 1) A file descriptor of libcurl is not matched to any easy handles. We can get a file descriptor from curl_multi_fdset(), we do not guarantee that it is a socket of easy handle just performed. If libcurl multi handle uses pipeline,

[E-devel] [Patch][elm_map] Bug fix: use safe iterator instead

2011-11-10 Thread Bluezery
Hi, There are bugs that map removes list elements when using EINA_LIST_FOREACH or EINA_LIST_REVERSE_FOREACH. We should use EINA_LIST_FOREACH_SAFE or EINA_LIST_REVERSE_FOREACH_SAFE instead for above case. Please review belows Index: src/lib/elm_map.c

Re: [E-devel] [Patch][ecore_con] Refactoring curl port

2011-11-11 Thread Bluezery
It's my mistake. fixed patch is attached again. I have removed active flag also. I use eina_list_data_find() instead. On Fri, Nov 11, 2011 at 4:54 PM, Sebastian Dransfeld s...@tango.flipp.net wrote: On 11/11/2011 03:09 AM, Bluezery wrote: Dear all, I have modified the way curl used

Re: [E-devel] [Patch][ecore_con] Refactoring curl port

2011-11-11 Thread Bluezery
warnnings because file descriptors cannot be controlled by ecore_con and only curl controls those internally. I think it can be ignored On Fri, Nov 11, 2011 at 5:55 PM, Sebastian Dransfeld s...@tango.flipp.net wrote: On 11/11/2011 09:41 AM, Bluezery wrote: It's my mistake. fixed patch is attached

Re: [E-devel] [Patch][ecore_con] Refactoring curl port

2011-11-11 Thread Bluezery
perform curl handling directly? S. On 11/11/2011 11:12 AM, Bluezery wrote: I have done two more things. Please check attached patch. I have tested using elm_map. I works well. 1) EINA_LIST_FOREACH_SAFE is removed and EINA_LIST_FREE is used instead. 2) I removed complete idler and use

[E-devel] [Patch][elm_map] Remove unnecessary flags

2011-11-11 Thread Bluezery
Hi, There are too may flags in elm_map. Most of them are duplicated and unnecessary. I have removed want, download, have variables. I use gi-job instead of want download. I use ecore_file_exists() instead of have flags. Please review attached patch file. Index: src/lib/elm_map.c

Re: [E-devel] [Patch][elm_map] Remove unnecessary flags

2011-11-12 Thread Bluezery
Previous patch has a bug. Fixed patch is attached. I have changed as below. // _process_download_list(gi-wd-obj); -- _process_download_list(gi-wd-obj); On Sat, Nov 12, 2011 at 3:51 PM, Bluezery ohpo...@gmail.com wrote: Hi, There are too may flags in elm_map. Most of them are duplicated

Re: [E-devel] [Patch][elm_map] Remove unnecessary flags

2011-11-13 Thread Bluezery
Are you give minutes to review this one?? :) On Sat, Nov 12, 2011 at 5:24 PM, Bluezery ohpo...@gmail.com wrote: Previous patch has a bug. Fixed patch is attached. I have changed as below. //  _process_download_list(gi-wd-obj);  -- _process_download_list(gi-wd-obj); On Sat, Nov 12, 2011

Re: [E-devel] [Patch][ecore_con] Refactoring curl port

2011-11-13 Thread Bluezery
This patch is rejected?? or could be in ?? :-) On Sat, Nov 12, 2011 at 3:05 AM, Cedric BAIL cedric.b...@free.fr wrote: Yop, On Fri, Nov 11, 2011 at 9:55 AM, Sebastian Dransfeld s...@tango.flipp.net wrote: On 11/11/2011 09:41 AM, Bluezery wrote: It's my mistake. fixed patch is attached

Re: [E-devel] [Patch][ecore_con] Refactoring curl port

2011-11-14 Thread Bluezery
Sebastian Dransfeld s...@tango.flipp.net wrote: On 11/14/2011 01:47 AM, Bluezery wrote: This patch is rejected?? or could be in ?? :-) To me it seems fine, but I don't want to commit it now so close before release of 1.1. Either someone how knows ecore_con better than me (Cedric

[E-devel] [Patch][ecore_file] Fix buffer flush bug

2011-11-24 Thread Bluezery
Dear all, fclose() should be called before callback is called to notify user of download completion because it cannot be flushed sometimes. Please review this patch. Index: src/lib/ecore_file/ecore_file_download.c === ---

[E-devel] [Patch][elm_map] Fix memory leak Add error handling

2011-11-27 Thread Bluezery
Dear all, I have done 3 jobs in this patch (1) I have fixed memory leaks in _nominatim_url_cb() function. (2) Add error handling to see the definite error log because sometimes evas_object_image_file_set() is failed. (3) Move zoom_min_get()/zoom_max_get() to source_init() to simplify code.

Re: [E-devel] [Patch][elm_map] Fix memory leak Add error handling

2011-11-27 Thread Bluezery
Hi, Zoom max/min value is changed by using wd-src-zoom in elm_map_source_name_set() function. I think there are no problem :-) On Mon, Nov 28, 2011 at 4:25 PM, Kim Yunhan spb...@gmail.com wrote: If the map source is changed, this patch seems that zoom_min and zoom_max is not changed. Could you

Re: [E-devel] [Patch][elm_map] Fix memory leak Add error handling

2011-11-28 Thread Bluezery
zoom level to 12. But wd-zoom_max is not changed to 12. It will be caused another problem. On Mon, Nov 28, 2011 at 4:44 PM, Bluezery ohpo...@gmail.com wrote: Hi, Zoom max/min value is changed by using wd-src-zoom in elm_map_source_name_set() function. I think there are no problem

Re: [E-devel] [Patch][elm_map] Fix memory leak Add error handling

2011-11-28 Thread Bluezery
, 2011 2:10 PM, Bluezery ohpo...@gmail.com wrote: Dear all, I have done 3 jobs in this patch (1) I have fixed memory leaks in _nominatim_url_cb() function. (2) Add error handling to see the definite error log because sometimes evas_object_image_file_set() is failed. (3) Move zoom_min_get

[E-devel] [Patch][elm_map] Fix OSM URL Bug

2011-11-30 Thread Bluezery
Dear all, I have fixed osm url for acquiring map tiles correctly. I have added Mapquest Mapquest open aerial map sources. I have referred http://wiki.openstreetmap.org/wiki/Slippy_map_tilenames for fixing and adding new url. Mapint is deprecated, so I remove it. Please refer

[E-devel] [Patch][test_map] Fix invalid object and logs

2011-12-02 Thread Bluezery
Dear all, I have using elementary_test for testing elementary map widget. But it use invalid object for signal callback. I have changed win -- map. Also I have fixed invalid log message. Please review this patch. -- BRs, Kim. Index: src/bin/test_map.c

Re: [E-devel] [Patch][elm_map] Fix OSM URL Bug

2011-12-02 Thread Bluezery
. On Thu, Dec 1, 2011 at 11:21 AM, Bluezery ohpo...@gmail.com wrote: Dear all, I have fixed osm url for acquiring map tiles correctly. I have added Mapquest Mapquest open aerial map sources. I have referred http://wiki.openstreetmap.org/wiki/Slippy_map_tilenames for fixing and adding new url

Re: [E-devel] [Patch][ecore_con] Refactoring curl port

2011-12-04 Thread Bluezery
Hi, Congratulations on EFL1.1 release!!! Now, Can it be added to svn?? :) 2011/11/15 Carsten Haitzler ras...@rasterman.com: On Mon, 14 Nov 2011 18:47:08 +0900 Bluezery ohpo...@gmail.com said: considering the amount of refactoring.. this will have to wait until after 1.1. we are in feature

Re: [E-devel] [Patch][elm_map] Fix OSM URL Bug

2011-12-05 Thread Bluezery
and attach patch again. However, elm_map is not maintained?? It is hard to be reviewed and be patched to map... ;-( I think elementary map has some heisenbugs and design issues. On Thu, Dec 1, 2011 at 11:21 AM, Bluezery lt;ohpo...@gmail.comgt; wrote: Dear all, I have fixed osm url for acquiring

[E-devel] [Patch][elm_map] Change grid management

2011-12-05 Thread Bluezery
Hi, There are no caching mechanism in current elm_map. So, too many network resources are wasted because elm_map keeps only two grids and already downloaded images are downloaded again and again. (This also slows the map loading speed) I have changed this grid management policy. I have done

Re: [E-devel] [Patch][elm_map] Change grid management

2011-12-12 Thread Bluezery
Dear, all I am waiting for a week. But no one may be interested in map issues.. ;-( It will be my pleasure if someone reviews this patch :-) 2011/12/6 Bluezery ohpo...@gmail.com: Hello, I separate my path into 4 patches. It should be patched sequentially (1) Grid Management

Re: [E-devel] [Patch][elm_map] Change grid management

2011-12-12 Thread Bluezery
Yeah, It's my mistake! I remade my patches to be applied sequentially. Those also reflect hermet's reviews. :-) 2011/12/13 Carsten Haitzler ras...@rasterman.com: On Tue, 6 Dec 2011 14:05:02 +0900 Bluezery ohpo...@gmail.com said: patch 01 applies. patch 02 seems already partly in 01

[E-devel] [Patch][elm_map] Bug fix for map test

2011-12-13 Thread Bluezery
Hello, This patch is for test map. This fixes two simple bugs. 1. Max zoom level can be changed depends on the map source, not always 18. 2. $HOME can be NULL. Thanks~ -- BRs, Kim. Index: src/bin/test_map.c === ---

[E-devel] elm gesture layer strange behavior

2011-12-13 Thread Bluezery
Dear, all Currently I am changing multi touch zoom to be used elm_gesture. But, there may be some strange behaviors when using elm_gesture. I looked around codes... and I found belows... ELM_GESTURE_STATE_START call back is always called when multi touch down event occurred. But

[E-devel] [Patch][elm_config] Add pinch zoom enable configuration

2011-12-14 Thread Bluezery
Dear all, Elementary can be used for all because EFL is an open source. But multi Touch event can have paten issue such as pinch zooming action. So, I add finger(pinch) zooming action to elm_config to can be enabled or disabled. This configuration value can be used for diverse product. It's my

Re: [E-devel] [Patch][elm_config] Add pinch zoom enable configuration

2011-12-15 Thread Bluezery
2011/12/15 Tom Hacohen tom.haco...@partner.samsung.com: On 15/12/11 08:11, Daniel Juyung Seo wrote: Overall idea looks good. 1) wrap 80 chars evas_object_event_callback_add(wd-rect, EVAS_CALLBACK_RESIZE, _rect_resize_cb, obj); This is too long. EFL sources are written based on 80 chars

[E-devel] [Patch][ecore_con_url] Add proxy timeout set API

2011-12-16 Thread Bluezery
Dear all, There existed two problems when using ecore_con_url. We cannot set proxy and set timeout. So, I added two APIs for solving these problems. Proxy can be set by setting libcurl option. Timeout also can be set but It need to add handler for Linux alarm signal. (Please refer:

Re: [E-devel] [Patch][ecore_con_url] Add proxy timeout set API

2011-12-16 Thread Bluezery
2011/12/16 Carsten Haitzler ras...@rasterman.com: why do we need to set proxy - curl inherits $http_proxy and $https_proxy env vars so it will use the system proxy settings... ? Yes, environment value can do that in normal environment. But each application may want to use it's own proxy. For

Re: [E-devel] [Patch][ecore_con_url] Add proxy timeout set API

2011-12-16 Thread Bluezery
=== --- ChangeLog (리비전 66269) +++ ChangeLog (작업 사본) @@ -407,3 +407,7 @@ * Add ability to get resource id of last x error in ecore_x. +2011-12-16 Tae-Hwan Kim (Bluezery) + + * Add proxy set and timeout set functions in ecore_con

[E-devel] [Patch][elm_map] Apply elm gesture

2011-12-19 Thread Bluezery
Dear all, elm_map implement pinch gesture by using evas mouse event. But it made codes too complicated and dirty. So I have changed pinch gesture by using elm_gesture. wheel zooming is also improved somewhat. From now on, configuration change related elm_gesture easily applied to elm_map and we

Re: [E-devel] [Patch][ecore_con_url] Add proxy timeout set API

2011-12-19 Thread Bluezery
Hi, I merged with recent revision. Please review this :p. 2011/12/16 Bluezery ohpo...@gmail.com: I fixed doc ChangeLog News. 2011/12/16 Carsten Haitzler ras...@rasterman.com: if they want to dynamically adjust the feature should be in ecore_con to track the changes and adjust

[E-devel] [Patch][elm_config] Add pinch rotate enable configuration

2011-12-19 Thread Bluezery
Hi, In addition to pinch zoom configuration, I have added pinch rotation configuration. It's default behavior is disabled as well as pinch zoom configuration. I need Tom's help again :p... Please review this patch. -- BRs, Kim. Index: src/lib/elm_config.c

Re: [E-devel] [Patch][elm_config] Add pinch zoom enable configuration

2011-12-20 Thread Bluezery
2011/12/20 Tom Hacohen tom.haco...@partner.samsung.com: On 20/12/11 12:27, Carsten Haitzler (The Rasterman) wrote: agreed. programmer should simply try and use pinch gestures and if he doesnt get any either its not possible (no multi-touch device) or, no emulated gesture/input is being used

Re: [E-devel] [Patch][ecore_con_url] Add proxy timeout set API

2011-12-20 Thread Bluezery
2011/12/20 Carsten Haitzler ras...@rasterman.com: On Mon, 19 Dec 2011 18:07:10 +0900 Bluezery ohpo...@gmail.com said: ecore_con_url_proxy_set() should return Eina_Bool (EINA_TRUE for success, EINA_FALSE for failure). I implemented ecore_con_url_proxy_set() similar to ecore_con_url_ssl_ca_set

Re: [E-devel] [Patch][elm_config] Add pinch zoom enable configuration

2011-12-20 Thread Bluezery
2011/12/21 Carsten Haitzler ras...@rasterman.com: On Wed, 21 Dec 2011 11:44:29 +0900 Bluezery ohpo...@gmail.com said: 2011/12/20 Tom Hacohen tom.haco...@partner.samsung.com: On 20/12/11 12:27, Carsten Haitzler (The Rasterman) wrote: agreed. programmer should simply try and use pinch

[E-devel] [Patch][ecore_con] Read ECORE_CON_SOCKS_V4 for ecore_con_url

2011-12-21 Thread Bluezery
Hi, I have heard that ECORE_CON_SOCKS_V4 is used for socks proxy. So, I add to read this value when econ_con_url is used. But I can not test this. because I cannot access socks proxy server. @discomfitor can help to review this patch. -- BRs, Kim.

Re: [E-devel] [Patch][ecore_con] Read ECORE_CON_SOCKS_V4 for ecore_con_url

2011-12-21 Thread Bluezery
2011/12/21 Michael Blumenkrantz michael.blumenkra...@gmail.com: On Wed, 21 Dec 2011 17:42:22 +0900 Bluezery ohpo...@gmail.com wrote: Hi, I have heard that ECORE_CON_SOCKS_V4 is used for socks proxy. So, I add to read this value when econ_con_url is used. But I can not test this. because I

[E-devel] [Patch][elm_map] Mouse wheel bug fix

2011-12-21 Thread Bluezery
Hi, There is mouse wheel bug. Actually I have made it in previous patch :p When using mouse wheel, tile size was changed between 0.5 ~ 2.0 before zoom level is changed. But actually it should be changed between 1.0 ~ 2.0. I have fixed it. If tile size is below 1.0, it does zoom-out. And if

Re: [E-devel] [Patch][ecore_con] Read ECORE_CON_SOCKS_V4 for ecore_con_url

2011-12-21 Thread Bluezery
2011/12/22 Michael Blumenkrantz michael.blumenkra...@gmail.com: On Thu, 22 Dec 2011 12:32:36 +0900 Bluezery ohpo...@gmail.com wrote: 2011/12/21 Michael Blumenkrantz michael.blumenkra...@gmail.com: On Wed, 21 Dec 2011 17:42:22 +0900 Bluezery ohpo...@gmail.com wrote: Hi, I have heard

Re: [E-devel] elementary.h split?

2011-12-21 Thread Bluezery
2011/12/22 Jihoon Kim imfin...@gmail.com: Current Elementary.h is too big. I'd like to vote for #1 (Separate headers by widget: list, genlist, gengrid, actionslider, ctxpopup, ...) . On Thursday, December 22, 2011, Daniel Juyung Seo wrote: Yay! You changed your mind? We already had some

[E-devel] [Patch][elm_map] Fix mouse down event propagation

2011-12-22 Thread Bluezery
Hi, It's simple bug fix for propagate mouse down event via long press event properly. This also is from me :p.. -- BRs, Kim. Index: src/lib/elm_map.c === --- src/lib/elm_map.c (리비전 66463) +++ src/lib/elm_map.c (작업 사본) @@ -1609,6

[E-devel] [Patch][elm_map] Bug fix for elm_map_geo_region_get()

2011-12-26 Thread Bluezery
Dear all, I have found a bug that calculation fails when view port size is bigger than map size. For example, if map size is 256x256 but view port size is 798x798 (when zoom level is a 1), center longitude latitude is out bound from normal longitude latitude. I have fixed it. Please review

[E-devel] [Patch][elm_map] Pause progressbar animator when map is paused

2011-12-26 Thread Bluezery
Hi, When map is paused by using elm_map_paused_set(), all existing animations is stopped. (ref: http://docs.enlightenment.org/auto/elementary/group__Map.html#ga6ab43ef6429140f8cccbd3669c5ed52e) But progress bar can be still running before all download requests are completed. It is equal to user

[E-devel] [Patch][elm_map] Smooth stepping of wheel pinch gesture

2011-12-28 Thread Bluezery
Hi, When elm_map doing zooming by using wheel gesture, scroller is resized if it exceeds certain limit. But current limit value is 2.0 1.0, actually those should be below 2.0 and above 1.0. The reason is that resizing means exactly 2.0 or 1.0. So if it is above 1.99x, it can be 2.0 and can be

[E-devel] [Patch][elm_map] Fix zoom rotated coordinates bug

2012-01-02 Thread Bluezery
Hi, In my point of view, elm_map did not consider zooming rotation by using Evas_Map initially. There are no problems if you use only elm_map_zoom_set(). But if you use pinch gesture or wheel to do subtle zooming rotating, the calculations of coordinates are not correct currently. I have fixed

[E-devel] [Patch][ecore_con] Fix return value checking

2012-01-02 Thread Bluezery
Hi, There is invalid return value checking of curl_multi_perform() in ecore_con_url. Because CURLM_CALL_MULTI_PERFORM is also not CURLM_OK, CURLM_CALL_MULTI_PERFORM checking should precede CURLM_OK checking. It's simple fix. Please review this -- BRs, Kim. Index:

[E-devel] Elementary Map Overlay External support

2012-01-08 Thread Bluezery
Dear all, I am interested in elementary map and working on it after past 2~3 months. I found some issues to be discussed with EFL developers. 1) Overlay Support For App. developers (users), elm_map only supports marker overlay. If users want to overlay line or polygon, users should make it'

Re: [E-devel] [Patch][elm_map] Fix zoom rotated coordinates bug

2012-01-09 Thread Bluezery
. So, I have modified patches. Anyway, no one may interested in elm_maps... Please review this patchplz... 2012/1/2 Bluezery ohpo...@gmail.com: Hi, In my point of view, elm_map did not consider zooming rotation by using Evas_Map initially. There are no problems if you use only

Re: [E-devel] [Patch][elm_map] Fix zoom rotated coordinates bug

2012-01-10 Thread Bluezery
Carsten Haitzler ras...@rasterman.com: On Mon, 9 Jan 2012 17:28:21 +0900 Bluezery ohpo...@gmail.com said: sorry - been busy for the past week or so and havent had time to deal with my backlog. i'm onto it now. comment:  elm_map_convert_canvas_into_geo() - should rename to be more efl strict oo

Re: [E-devel] Elementary Map Overlay External support

2012-01-11 Thread Bluezery
I think that no replay means I am allowed to do this~!! hahaha... Anyway, this will make huge changes in elm_map. I look forward that someone will review my changes. :D 2012/1/9 Bluezery ohpo...@gmail.com: Dear all, I am interested in elementary map and working on it after past 2~3 months

Re: [E-devel] large ecore-con-url bug

2012-02-17 Thread Bluezery
Actually, ecore_con_url_new(http://;) should be used fore one url. Also it returns one handler. for this URL. But, if you need not this handler, you can discard handler after ecore_con_url_get() is called. I have fixed attached examples and it works well. :D I think that you did not use

Re: [E-devel] large ecore-con-url bug

2012-02-17 Thread Bluezery
michael.blumenkra...@gmail.com: On Sat, 18 Feb 2012 14:05:54 +0900 Bluezery ohpo...@gmail.com wrote: I have attached a log file of your example code. It do download both urls. I think that there are differences between you libcurl and my libcurl version.  My libcurl version is 7.21.6. 2012/2

Re: [E-devel] large ecore-con-url bug

2012-02-17 Thread Bluezery
It is hard to find the problems. I cannot reproduce the problem. :( Maybe your test environment and mine are different. Can anyone test this example??? 2012/2/18 Michael Blumenkrantz michael.blumenkra...@gmail.com: On Sat, 18 Feb 2012 14:45:00 +0900 Bluezery ohpo...@gmail.com wrote: Um

Re: [E-devel] [Patch][elm_map] Refactoring Elementary Map

2012-02-18 Thread Bluezery
.) But we can't review this big patch at a time. Please split this patch as small as possible. Thanks. Daniel Juyung Seo (SeoZ) On Thu, Feb 16, 2012 at 8:14 PM, Bluezery ohpo...@gmail.com wrote: Hello, I have done refactoring code clean-up in elm_map.  Event if this patch have much changes

Re: [E-devel] large ecore-con-url bug

2012-02-18 Thread Bluezery
Yeah, It may be caused by system difference. It's hard to find problems. But eocre_con_url should run independently. I have made libcurl testing application. It's almost same implementation as ecore_con_url's stuff. Can you test attached curl test ? when testing it, redirect standard out to

Re: [E-devel] large ecore-con-url bug

2012-02-18 Thread Bluezery
OMG.. I attach my curl_test log. Can I get the your log ?? 2012/2/19 Michael Blumenkrantz michael.blumenkra...@gmail.com: On Sun, 19 Feb 2012 15:32:56 +0900 Bluezery ohpo...@gmail.com wrote: Yeah, It may be caused by system difference. It's hard to find problems. But eocre_con_url should

Re: [E-devel] [Patch][elm_map] Refactoring Elementary Map

2012-02-20 Thread Bluezery
I fixed the coding convention as your suggestions. And actually I did not touched almost those codes related with track route yet :D. Those will be fixed later ASAP. Please review this formatting patch. 2012/2/20 Carsten Haitzler ras...@rasterman.com: On Thu, 16 Feb 2012 20:14:25 +0900 Bluezery

[E-devel] [Patch][ecore_con] ecore_con_url Bug fix Add timeout check

2012-02-20 Thread Bluezery
Hello, Previously discomfitor complained about ecore_con_url' bug. I think that It comes from environment differences and relates with timing. I found something can be a solution. In previous ecore_con_url, if no file descriptors are set, It waits forever. libcurl sometimes do not set these

[E-devel] [Patch][elm_map] Emap enable patch

2012-02-21 Thread Bluezery
Hello, Previous my giant patches made build errors when EMAP is enabled. This patch fixed this. 'ox' 'oy' is almost useless values (-1 or 0 value). I will fix and upgrade all things related EMAP later. Please review this patch -- BRs, Kim. Index: src/lib/elm_map.c

[E-devel] [Patch][elm_map] Map gesture rotate fix

2012-02-21 Thread Bluezery
Hello, Previsouly i have tested rotation with old version of gesture layer in touch devices. Recent gesture layer gives somewhat different event information. So I fixed it accordingly. I tested this patch in touch-enabled device. Please review ths patch. -- BRs, Kim. Index: src/lib/elm_map.c

[E-devel] [Patch][ecore_con] Fix invalid curl handle removal by valgrind

2012-02-22 Thread Bluezery
Hello, discomfitor reports bugs by valigrind. I checked it with valgrid and I found the clues curl_multi_remove_handle() should not be called when multi handles being performed. So I removed curl_multi_remove_handle() code from _ecore_con_url_info_read() Now, curl_multi_remove_handle() is only

Re: [E-devel] ecore_file_download() always abort after 30 seconds

2012-02-26 Thread Bluezery
2012/2/27 Davide Andreoli d...@gurumeditation.it: 2012/2/26 Carsten Haitzler ras...@rasterman.com On Sat, 25 Feb 2012 22:50:59 +0100 Davide Andreoli d...@gurumeditation.it said: Please give a try at ecore/src/examples/ecore_file_download_example.c All the download will fail after 30

Re: [E-devel] Several elementary API renames

2012-03-01 Thread Bluezery
2012/3/1 Pau Espin Pedrol pespin.s...@gmail.com: Hi  everybody, I'd like to introduce some little API elementary rename changes to add more consistency with the rest of the API and makes it easier to write bindings for them. elm_map.h: Elm_Map_Route_Sources - Elm_Map_Route_Source

[E-devel] (no subject)

2012-03-02 Thread Bluezery
Hello, This patch is for reading socks version5 proxy from global variables. This is almost same as previous socks version4. discomfitor can help to review this patch. Thanks. -- BRs, Kim. Index: src/lib/ecore_con/ecore_con_url.c

Re: [E-devel] (no subject)

2012-03-03 Thread Bluezery
2012/3/3 Michael Blumenkrantz michael.blumenkra...@gmail.com: On Sat, 3 Mar 2012 13:19:24 +0900 Bluezery ohpo...@gmail.com wrote: Hello, This patch is for reading socks version5 proxy from global variables. This is almost same as previous socks version4. discomfitor can help to review

Re: [E-devel] [Patch][elm_map] Add overlay in the map

2012-03-04 Thread Bluezery
2012/2/28 Bluezery ohpo...@gmail.com: The patch file of the previous mail has some build warnings bubble example fix. I removed those. 2012/2/28 Bluezery ohpo...@gmail.com: Hello, I made a overlay APIs in the map. This will replace the marker APIs. I hope that markers will be deleted

Re: [E-devel] elm function pointers as args

2012-03-06 Thread Bluezery
Elm_Map_Marker_XXX_Func() will be deprecated if my previous patch is in. But Elm_Map_Module_XXX_Func() are module interfaces for plug-in developers. In EFL, these are not necessary to be revealed in the header file? or these are moved to separated plug-in header for 3rd party plug-in developer?

Re: [E-devel] [Patch][elm_map] Change Add map APIs

2012-03-06 Thread Bluezery
Hello, This patch being reviewd?? I will appreciate if this patch will be in. :) 2012/3/5 David Seikel onef...@gmail.com: On Mon, 5 Mar 2012 20:05:51 +0900 Bluezery ohpo...@gmail.com wrote: Hello, I rename add some APIs. As previously discussed, I categorized map APIs. (http://marc.info

[E-devel] [Patch][elm_map] Fix warning Add a API

2012-03-07 Thread Bluezery
Hello, These can be final patch before elementary 1.0 release. 1) As raster's comment, I removed plug-in function prototypes and removed deprecated APIs in test_map.c 2) Add elm_map_region_to_canvas_convert(). This is reverse of elm_map_canvas_to_region_convert() 3) Remove all _geo_ in even

[E-devel] [patch] elm_map - Bug fix

2012-03-10 Thread Bluezery
Hello, This patch fixes two elm map bugs 1) Geometry calculation error Calculation of canvas geometry is not correct If other evas objects take up some space in the canvas. This patch fix this by calculating smart pan geometry values. 2) Map moves southeastward when zooming When repeating zoom

[E-devel] [Patch][ecore_con] Simple bug fix

2012-03-11 Thread Bluezery
Hello, This is patch to prevent a segfault if curlmsg == NULL. Thanks. -- BRs, Kim. Index: src/lib/ecore_con/ecore_con_url.c === --- src/lib/ecore_con/ecore_con_url.c (리비전 69188) +++ src/lib/ecore_con/ecore_con_url.c (작업 사본) @@

[E-devel] [Patch][enki] Fix build breaks related elm_map

2012-03-12 Thread Bluezery
Hello, enki cannot be built because of map API changes. I fixed those build warnings breaks related with elm_map. I know that J.H. Park currently doing this jobs. This patch can help her :D. Please review this patch. Thanks. -- BRs, Kim. enki@fix_map@patch Description: Binary data

Re: [E-devel] [Patch][enki] Fix build breaks related elm_map

2012-03-12 Thread Bluezery
); and some else if branches removed. I hope you have verified your changes. Kindly confirm, make the required changes and resubmit. Regards, Sanjeev On Mon, Mar 12, 2012 at 7:09 PM, Bluezery ohpo...@gmail.com wrote: Hello, enki cannot be built because of map API changes. I fixed

Re: [E-devel] [patch] elm_map - Bug fix

2012-03-17 Thread Bluezery
2012/3/10 Bluezery ohpo...@gmail.com: Hello, This patch fixes two elm map bugs 1) Geometry calculation error Calculation of canvas geometry is not correct If other evas objects take up some space in the canvas. This patch fix this by calculating smart pan geometry values. 2) Map moves

Re: [E-devel] [patch] elm_map - Bug fix

2012-03-18 Thread Bluezery
2012/3/19 Sanjeev BA eflel...@gmail.com: I am looking into it. In removed/deprecated header, some APIs are removed. So those are not need to be patched. I resend modified patch. Thanks. On Sun, Mar 18, 2012 at 12:08 PM, Bluezery ohpo...@gmail.com wrote: 2012/3/10 Bluezery ohpo...@gmail.com

Re: [E-devel] [patch] elm_map - Bug fix

2012-03-20 Thread Bluezery
2012/3/19 Bluezery ohpo...@gmail.com: 2012/3/19 Sanjeev BA eflel...@gmail.com: I am looking into it. In removed/deprecated header, some APIs are removed. So those are not need to be patched. I resend modified patch. Thanks. Dear Sanjeeve, I am waiting for you. :) On Sun, Mar 18, 2012

Re: [E-devel] [patch] elm_map - Bug fix

2012-03-20 Thread Bluezery
I resend map patch! Thanks for your comments! ! 2012/3/20 Bluezery ohpo...@gmail.com: 2012/3/19 Bluezery ohpo...@gmail.com: 2012/3/19 Sanjeev BA eflel...@gmail.com: I am looking into it. In removed/deprecated header, some APIs are removed. So those are not need to be patched. I resend

Re: [E-devel] [Patch][elm_map] Map overlay color grouping route Fix

2012-03-21 Thread Bluezery
On Wed, 21 Mar 2012 15:59:32 +0900 Daniel Juyung Seo seojuyu...@gmail.com said: Actually slow review is a bad thing happening in EFL. I need more reviewers. But big patch makes review even slower and you won't get correct review for your big patch. Just separate your job. Do not split the

[E-devel] [Patch][elm_map] Remove unused codes

2012-03-22 Thread Bluezery
Hello, Now, Elm_Map_Marker, Elm_Map_Marker_Class, Elm_Map_Group_Class, Marker_Bubble are not used anymore. So I removed all things related those in elm_map. Please review this patch. Thanks -- BRs, Kim. Index: src/lib/elm_map.c

[E-devel] [Patch][elm_map] Bug fix

2012-03-22 Thread Bluezery
Hello, This is simple bug fix patch Callback parameter should be Widget_Data, not Evas_Object. Thanks. -- BRs, Kim. Index: src/lib/elm_map.c === --- src/lib/elm_map.c (리비전 69574) +++ src/lib/elm_map.c (작업 사본) @@ -4284,9 +4284,9 @@

[E-devel] [Patch][elm_map] Fix related zoom

2012-03-23 Thread Bluezery
Hello, This is patch for zoom behavior in elm_map. 1) wd-zoom_max is defined by user. wd-src_tile-zoom_max is defined by source map provider. Therefore, I decide to do not touch those things. Just check these while zooming ( zoom_do() ) 2) When pan being re-sized, I changed from zoom with

Re: [E-devel] [Patch][elm_map] Fix related zoom

2012-03-23 Thread Bluezery
2012/3/23 Sanjeev BA eflel...@gmail.com: Why is zoom with animation changed to zoom_do ? Because too many animations occurred while window size is changing. :) Zoom with animation can be used by elm_map_zoom_set(). On Fri, Mar 23, 2012 at 7:36 PM, Bluezery ohpo...@gmail.com wrote: Hello

Re: [E-devel] [PATCH] ecrire:cfg.c,main.c malloc return checks

2012-03-27 Thread Bluezery
2012/3/27 Vikram Narayanan vikram...@gmail.com: On 3/27/2012 12:37 AM, Michael Blumenkrantz wrote: On Tue, 27 Mar 2012 00:21:19 +0530 Vikram Narayananvikram...@gmail.com  wrote: In ecrire, Check return values of *alloc functions, though its unlikely to fail. There are chances though!

[E-devel] [Patch][eina_module] Add eina_module_resident() API

2012-03-30 Thread Bluezery
Hello, I want to add new eina_module_resident() API in eina_modules. This function makes the module never be unloaded before process is closed. This is needed in which case a module should not be closed. For example, if a module uses static GObject, closing and reopening it cause crashes.

Re: [E-devel] [Patch][eina_module] Add eina_module_resident() API

2012-03-31 Thread Bluezery
2012/3/30 Gustavo Sverzut Barbieri barbi...@profusion.mobi: On Fri, Mar 30, 2012 at 3:43 AM, Bluezery ohpo...@gmail.com wrote: Hello, I want to add new eina_module_resident() API in eina_modules. This function makes the module never be unloaded before process is closed. This is needed

[E-devel] [Patch][eina] Add -ldl to eina.pc

2012-04-02 Thread Bluezery
Hello, This patch adds -ldl to Libs.private in eina.pc If libraries are internally used and not recognized by pkg-config, those should be added to Libs.private. It can be used when static linking. Thanks. -- BRs, Kim. eina@fix_pc.patch Description: Binary data

Re: [E-devel] [Patch][eina_module] Add eina_module_resident() API

2012-04-02 Thread Bluezery
2012/4/2 Carsten Haitzler ras...@rasterman.com: On Sat, 31 Mar 2012 21:33:50 +0900 Bluezery ohpo...@gmail.com said: 2012/3/30 Gustavo Sverzut Barbieri barbi...@profusion.mobi: On Fri, Mar 30, 2012 at 3:43 AM, Bluezery ohpo...@gmail.com wrote: Hello, I want to add new

Re: [E-devel] [Patch][eina] Add -ldl to eina.pc

2012-04-03 Thread Bluezery
. Vincent On Mon, Apr 2, 2012 at 10:31 AM, Bluezery ohpo...@gmail.com wrote: Hello, This patch adds -ldl to Libs.private in eina.pc If libraries are internally used and not recognized by pkg-config, those should be added to Libs.private. It can be used when static linking. Thanks. -- BRs, Kim

  1   2   >