[snip]
>> OK.
>> Checking the boundaries after _advance_after_end_of_string() is more 
>> expensive
>> than removing the last \0 or reducing the map_end by one.
>>
>> How could one make the fix upstream?  Keeping my own offline patches
>> would be a maintenance
>> nightmare.
>
> I'll go with the check after map since it's safer and people extending
> Evas would not break it accidentally since they removed or added \0 at
> the end of the string.   The integer comparison is not hat heavy
> anyway.

OK.

>
> See if this applies and fixes the bug. Sorry for the long back and
> forth with this thread :-(

Applied, tried and yes, it fixes the bug.
It's ok for discussions as long as we get to fix the bug. :-)

Thanks.


brian

>
>
> Index: src/lib/canvas/evas_object_textblock.c
> ===================================================================
> --- src/lib/canvas/evas_object_textblock.c      (revision 43746)
> +++ src/lib/canvas/evas_object_textblock.c      (working copy)
> @@ -2718,6 +2718,8 @@
>
>        escape = map_itr;
>        _advance_after_end_of_string(&map_itr);
> +       if (map_itr >= map_end) break;
> +
>        mc = map_itr;
>        sc = s;
>        match = 1;
>



>
>
> --
> Gustavo Sverzut Barbieri
> http://profusion.mobi embedded systems
> --------------------------------------
> MSN: barbi...@gmail.com
> Skype: gsbarbieri
> Mobile: +55 (19) 9225-2202
>



-- 
brian
------------------

Cool-Karaoke - The smallest recording studio, in your palm, open-sourced
http://cool-idea.com.tw/

iMaGiNaTiOn iS mOrE iMpOrTaNt tHaN kNoWlEdGe

------------------------------------------------------------------------------
Let Crystal Reports handle the reporting - Free Crystal Reports 2008 30-Day 
trial. Simplify your report design, integration and deployment - and focus on 
what you do best, core application coding. Discover what's new with
Crystal Reports now.  http://p.sf.net/sfu/bobj-july
_______________________________________________
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to