Re: [E-devel] [Patch][Evil] evil_string has strcasestr
On Fri, 8 Jul 2011 21:27:21 +0900 cnook said: sure - and as i said... if and when you have a widget ready that needs it... then re-submit the patch for evil - you'll be showing how it's used and needed. (or not) :) > Thanks for your response! > > I'm not sure other codes(languages? difference set?) need the strcasestr();. > Even though the strcasestr(); is meaningless, it would work as strstr(); > Anyhow, I will check strcasestr(); is really necessary or not. > > Have a good time! > > > 2011/7/5 Carsten Haitzler > > > On Fri, 1 Jul 2011 10:15:21 +0900 cnook said: > > > > now... that looks technically ok... but why are you using strcasestr. > > manual > > page is pretty specific: > > > > The strcasestr() function is a nonstandard extension. > > > > so it's non-standard and.. well mingw and other os's might definitely not > > have > > it. we tend to stick to standard functions as much as we can. > > > > if you want to put this patch in as PART of submitting a new widget - i'm > > happy > > to look at it then in the context of the new widget and why it needs it. > > you do > > know that you may have to make this unicode friendly depending on usage? it > > the > > code you have there does just ascii (or the ascii portion of utf8 - > > skipping > > toupper/lower of anything beyond ascii) > > > > > Hi :-] > > > > > > Your review comments in the attached patch. > > > Thanks for your response always. > > > > > > Sincerely, > > > Shinwoo Kim. > > > > > > 2011/6/29 Vincent Torri > > > > > > > > > > > > > > > On Wed, 29 Jun 2011, cnook wrote: > > > > > > > > Oh I see, I got the point. Thanks for your response. > > > >> > > > >> Wherever the strcasestr() is used using previous patch, would be > > changed > > > >> as > > > >> bellow. > > > >> > > > >> #ifdef HAVE_STRCASESTR > > > >> strcase(h, n); > > > >> #else > > > >> _elm_util_strcasestr(h, n); > > > >> #endif > > > >> > > > > > > > > better: > > > > > > > > #ifdef HAVE_STRCASESTR > > > > # define _elm_util_strcasestr(h, n) strcase((h), (n)) > > > > #endif > > > > > > > > and always use _elm_util_strcasestr() instead of strcasestr() > > > > > > > > Vincent > > > > > > > > > > > >> Anyhow, I have refined the previous patch and attached. > > > >> Then, wherever the strcasestr() is used, should be changed just as > > bellow > > > >> > > > >> _elm_util_strcasestr(h, n); > > > >> > > > >> > > > >> Sincerely, > > > >> Shinwoo Kim. > > > >> > > > >> > > > >> 2011/6/29 Vincent Torri > > > >> > > > >> > > > >>> what he means is: > > > >>> > > > >>> * using AC_CHECK_FUNCS([strcasestr]) in configure.ac defined > > > >>> HAVE_STRCASESTR in config.h > > > >>> * in the code, you use the macro HAVE_STRCASESTR to use the glibc > > > >>> version > > > >>> of strcase, or to use your own version > > > >>> > > > >>> Vincent > > > >>> > > > >>> > > > >>> On Wed, 29 Jun 2011, cnook wrote: > > > >>> > > > >>> Hi > > > >>> > > > >>>> > > > >>>> Thanks for your response. > > > >>>> I have no enough idea when it comes to E16. > > > >>>> Actually this patch comes because of the Evil Windows environments - > > > >>>> strcasestr does not exist. > > > >>>> If strcasestr alternative - Estrcasestr - is on the E16, Should I > > > >>>> install > > > >>>> the E16 also or not? > > > >>>> > > > >>>> Sincerely, > > > >>>> Shinwoo Kim. > > > >>>> > > > >>>> > > > >>>> 2011/6/29 Hyoyoung Chang > > > >>>> > > > >>>> I think it's a nice patch if glibc's strcasestr can be used > > > >>>> > > > >>>>&
Re: [E-devel] [Patch][Evil] evil_string has strcasestr
Thanks for your response! I'm not sure other codes(languages? difference set?) need the strcasestr();. Even though the strcasestr(); is meaningless, it would work as strstr(); Anyhow, I will check strcasestr(); is really necessary or not. Have a good time! 2011/7/5 Carsten Haitzler > On Fri, 1 Jul 2011 10:15:21 +0900 cnook said: > > now... that looks technically ok... but why are you using strcasestr. > manual > page is pretty specific: > > The strcasestr() function is a nonstandard extension. > > so it's non-standard and.. well mingw and other os's might definitely not > have > it. we tend to stick to standard functions as much as we can. > > if you want to put this patch in as PART of submitting a new widget - i'm > happy > to look at it then in the context of the new widget and why it needs it. > you do > know that you may have to make this unicode friendly depending on usage? it > the > code you have there does just ascii (or the ascii portion of utf8 - > skipping > toupper/lower of anything beyond ascii) > > > Hi :-] > > > > Your review comments in the attached patch. > > Thanks for your response always. > > > > Sincerely, > > Shinwoo Kim. > > > > 2011/6/29 Vincent Torri > > > > > > > > > > > On Wed, 29 Jun 2011, cnook wrote: > > > > > > Oh I see, I got the point. Thanks for your response. > > >> > > >> Wherever the strcasestr() is used using previous patch, would be > changed > > >> as > > >> bellow. > > >> > > >> #ifdef HAVE_STRCASESTR > > >> strcase(h, n); > > >> #else > > >> _elm_util_strcasestr(h, n); > > >> #endif > > >> > > > > > > better: > > > > > > #ifdef HAVE_STRCASESTR > > > # define _elm_util_strcasestr(h, n) strcase((h), (n)) > > > #endif > > > > > > and always use _elm_util_strcasestr() instead of strcasestr() > > > > > > Vincent > > > > > > > > >> Anyhow, I have refined the previous patch and attached. > > >> Then, wherever the strcasestr() is used, should be changed just as > bellow > > >> > > >> _elm_util_strcasestr(h, n); > > >> > > >> > > >> Sincerely, > > >> Shinwoo Kim. > > >> > > >> > > >> 2011/6/29 Vincent Torri > > >> > > >> > > >>> what he means is: > > >>> > > >>> * using AC_CHECK_FUNCS([strcasestr]) in configure.ac defined > > >>> HAVE_STRCASESTR in config.h > > >>> * in the code, you use the macro HAVE_STRCASESTR to use the glibc > > >>> version > > >>> of strcase, or to use your own version > > >>> > > >>> Vincent > > >>> > > >>> > > >>> On Wed, 29 Jun 2011, cnook wrote: > > >>> > > >>> Hi > > >>> > > >>>> > > >>>> Thanks for your response. > > >>>> I have no enough idea when it comes to E16. > > >>>> Actually this patch comes because of the Evil Windows environments - > > >>>> strcasestr does not exist. > > >>>> If strcasestr alternative - Estrcasestr - is on the E16, Should I > > >>>> install > > >>>> the E16 also or not? > > >>>> > > >>>> Sincerely, > > >>>> Shinwoo Kim. > > >>>> > > >>>> > > >>>> 2011/6/29 Hyoyoung Chang > > >>>> > > >>>> I think it's a nice patch if glibc's strcasestr can be used > > >>>> > > >>>>> > > >>>>> How about to add statements like below? > > >>>>> > > >>>>> #ifdef HAVE_STRCASESTR > > >>>>> #define Estrcasestr(haystack, needle) strcasestr(haystack, needle) > > >>>>> #else > > >>>>> const char *Estrcasestr(const char *haystack, const char > > >>>>> *needle); > > >>>>> #endif > > >>>>> > > >>>>> In E16/e/src/util.h > > >>>>> > > >>>>> > > >>>>> > > >>>>> From: cnook [mailto:kimci...@gmail.com] > > >>>>> > > >>>>>> Sent: Tuesday, June 28, 201
Re: [E-devel] [Patch][Evil] evil_string has strcasestr
On Fri, 1 Jul 2011 10:15:21 +0900 cnook said: now... that looks technically ok... but why are you using strcasestr. manual page is pretty specific: The strcasestr() function is a nonstandard extension. so it's non-standard and.. well mingw and other os's might definitely not have it. we tend to stick to standard functions as much as we can. if you want to put this patch in as PART of submitting a new widget - i'm happy to look at it then in the context of the new widget and why it needs it. you do know that you may have to make this unicode friendly depending on usage? it the code you have there does just ascii (or the ascii portion of utf8 - skipping toupper/lower of anything beyond ascii) > Hi :-] > > Your review comments in the attached patch. > Thanks for your response always. > > Sincerely, > Shinwoo Kim. > > 2011/6/29 Vincent Torri > > > > > > > On Wed, 29 Jun 2011, cnook wrote: > > > > Oh I see, I got the point. Thanks for your response. > >> > >> Wherever the strcasestr() is used using previous patch, would be changed > >> as > >> bellow. > >> > >> #ifdef HAVE_STRCASESTR > >> strcase(h, n); > >> #else > >> _elm_util_strcasestr(h, n); > >> #endif > >> > > > > better: > > > > #ifdef HAVE_STRCASESTR > > # define _elm_util_strcasestr(h, n) strcase((h), (n)) > > #endif > > > > and always use _elm_util_strcasestr() instead of strcasestr() > > > > Vincent > > > > > >> Anyhow, I have refined the previous patch and attached. > >> Then, wherever the strcasestr() is used, should be changed just as bellow > >> > >> _elm_util_strcasestr(h, n); > >> > >> > >> Sincerely, > >> Shinwoo Kim. > >> > >> > >> 2011/6/29 Vincent Torri > >> > >> > >>> what he means is: > >>> > >>> * using AC_CHECK_FUNCS([strcasestr]) in configure.ac defined > >>> HAVE_STRCASESTR in config.h > >>> * in the code, you use the macro HAVE_STRCASESTR to use the glibc > >>> version > >>> of strcase, or to use your own version > >>> > >>> Vincent > >>> > >>> > >>> On Wed, 29 Jun 2011, cnook wrote: > >>> > >>> Hi > >>> > >>>> > >>>> Thanks for your response. > >>>> I have no enough idea when it comes to E16. > >>>> Actually this patch comes because of the Evil Windows environments - > >>>> strcasestr does not exist. > >>>> If strcasestr alternative - Estrcasestr - is on the E16, Should I > >>>> install > >>>> the E16 also or not? > >>>> > >>>> Sincerely, > >>>> Shinwoo Kim. > >>>> > >>>> > >>>> 2011/6/29 Hyoyoung Chang > >>>> > >>>> I think it's a nice patch if glibc's strcasestr can be used > >>>> > >>>>> > >>>>> How about to add statements like below? > >>>>> > >>>>> #ifdef HAVE_STRCASESTR > >>>>> #define Estrcasestr(haystack, needle) strcasestr(haystack, needle) > >>>>> #else > >>>>> const char *Estrcasestr(const char *haystack, const char > >>>>> *needle); > >>>>> #endif > >>>>> > >>>>> In E16/e/src/util.h > >>>>> > >>>>> > >>>>> > >>>>> From: cnook [mailto:kimci...@gmail.com] > >>>>> > >>>>>> Sent: Tuesday, June 28, 2011 3:58 PM > >>>>>> To: Hyoyoung Chang > >>>>>> Cc: Daniel Juyung Seo; > >>>>>> enlightenment-devel@lists.**so**urceforge.net<http://sourceforge.net> > >>>>>> > >>>>>> > > >>>>>> > >>>>>> Subject: Re: [E-devel] [Patch][Evil] evil_string has strcasestr > >>>>>> > >>>>>> Dear All, > >>>>>> > >>>>>> Hi~ > >>>>>> Here is the patch added on the elementary. :-] > >>>>>> > >>>>>> Sincerely, > >>>>>> Shinwoo Kim. > >>>>>> 2011/6/22 Hyoyoung Chang > >>>>>> As I know strcasestr is not standard function. > >>>>>> It's a glib
Re: [E-devel] [Patch][Evil] evil_string has strcasestr
Hi :-] Your review comments in the attached patch. Thanks for your response always. Sincerely, Shinwoo Kim. 2011/6/29 Vincent Torri > > > On Wed, 29 Jun 2011, cnook wrote: > > Oh I see, I got the point. Thanks for your response. >> >> Wherever the strcasestr() is used using previous patch, would be changed >> as >> bellow. >> >> #ifdef HAVE_STRCASESTR >> strcase(h, n); >> #else >> _elm_util_strcasestr(h, n); >> #endif >> > > better: > > #ifdef HAVE_STRCASESTR > # define _elm_util_strcasestr(h, n) strcase((h), (n)) > #endif > > and always use _elm_util_strcasestr() instead of strcasestr() > > Vincent > > >> Anyhow, I have refined the previous patch and attached. >> Then, wherever the strcasestr() is used, should be changed just as bellow >> >> _elm_util_strcasestr(h, n); >> >> >> Sincerely, >> Shinwoo Kim. >> >> >> 2011/6/29 Vincent Torri >> >> >>> what he means is: >>> >>> * using AC_CHECK_FUNCS([strcasestr]) in configure.ac defined >>> HAVE_STRCASESTR in config.h >>> * in the code, you use the macro HAVE_STRCASESTR to use the glibc >>> version >>> of strcase, or to use your own version >>> >>> Vincent >>> >>> >>> On Wed, 29 Jun 2011, cnook wrote: >>> >>> Hi >>> >>>> >>>> Thanks for your response. >>>> I have no enough idea when it comes to E16. >>>> Actually this patch comes because of the Evil Windows environments - >>>> strcasestr does not exist. >>>> If strcasestr alternative - Estrcasestr - is on the E16, Should I >>>> install >>>> the E16 also or not? >>>> >>>> Sincerely, >>>> Shinwoo Kim. >>>> >>>> >>>> 2011/6/29 Hyoyoung Chang >>>> >>>> I think it's a nice patch if glibc's strcasestr can be used >>>> >>>>> >>>>> How about to add statements like below? >>>>> >>>>> #ifdef HAVE_STRCASESTR >>>>> #define Estrcasestr(haystack, needle) strcasestr(haystack, needle) >>>>> #else >>>>> const char *Estrcasestr(const char *haystack, const char >>>>> *needle); >>>>> #endif >>>>> >>>>> In E16/e/src/util.h >>>>> >>>>> >>>>> >>>>> From: cnook [mailto:kimci...@gmail.com] >>>>> >>>>>> Sent: Tuesday, June 28, 2011 3:58 PM >>>>>> To: Hyoyoung Chang >>>>>> Cc: Daniel Juyung Seo; >>>>>> enlightenment-devel@lists.**so**urceforge.net<http://sourceforge.net> >>>>>> >>>>>> > >>>>>> >>>>>> Subject: Re: [E-devel] [Patch][Evil] evil_string has strcasestr >>>>>> >>>>>> Dear All, >>>>>> >>>>>> Hi~ >>>>>> Here is the patch added on the elementary. :-] >>>>>> >>>>>> Sincerely, >>>>>> Shinwoo Kim. >>>>>> 2011/6/22 Hyoyoung Chang >>>>>> As I know strcasestr is not standard function. >>>>>> It's a glibc extension. >>>>>> >>>>>> CONFORMING TO >>>>>> The strstr() function conforms to C89 and C99. The strcasestr() >>>>>> funcÿÿ >>>>>> >>>>>> tion is a nonstandard extension. >>>>>> -- man page of strcasestr >>>>>> >>>>>> Is it only used in elementary? If then how about add to check routine >>>>>> in >>>>>> configure and add in elm_util.c? >>>>>> Or is it widely used in e project, it can be added in other module. >>>>>> >>>>>> PS. I google about strcasestr. It's contained in LSB 3.1 >>>>>> >>>>>> >>>>>> http://refspecs.freestandards.org/LSB_3.1.1/LSB-Core-** >>>>>> >>>>> generic/LSB-Core-generi>>>> LSB_3.1.1/LSB-Core-generic/**LSB-Core-generi<http://refspecs.freestandards.org/LSB_3.1.1/LSB-Core-generic/LSB-Core-generi> >>>>> > >>>>> >>>>> c/baselib-strcasestr.html >>>>>> >>>>>> >>>>>> -Original Message- >>>
Re: [E-devel] [Patch][Evil] evil_string has strcasestr
On Wed, 29 Jun 2011, cnook wrote: Oh I see, I got the point. Thanks for your response. Wherever the strcasestr() is used using previous patch, would be changed as bellow. #ifdef HAVE_STRCASESTR strcase(h, n); #else _elm_util_strcasestr(h, n); #endif better: #ifdef HAVE_STRCASESTR # define _elm_util_strcasestr(h, n) strcase((h), (n)) #endif and always use _elm_util_strcasestr() instead of strcasestr() Vincent Anyhow, I have refined the previous patch and attached. Then, wherever the strcasestr() is used, should be changed just as bellow _elm_util_strcasestr(h, n); Sincerely, Shinwoo Kim. 2011/6/29 Vincent Torri what he means is: * using AC_CHECK_FUNCS([strcasestr]) in configure.ac defined HAVE_STRCASESTR in config.h * in the code, you use the macro HAVE_STRCASESTR to use the glibc version of strcase, or to use your own version Vincent On Wed, 29 Jun 2011, cnook wrote: Hi Thanks for your response. I have no enough idea when it comes to E16. Actually this patch comes because of the Evil Windows environments - strcasestr does not exist. If strcasestr alternative - Estrcasestr - is on the E16, Should I install the E16 also or not? Sincerely, Shinwoo Kim. 2011/6/29 Hyoyoung Chang I think it's a nice patch if glibc's strcasestr can be used How about to add statements like below? #ifdef HAVE_STRCASESTR #define Estrcasestr(haystack, needle) strcasestr(haystack, needle) #else const char *Estrcasestr(const char *haystack, const char *needle); #endif In E16/e/src/util.h From: cnook [mailto:kimci...@gmail.com] Sent: Tuesday, June 28, 2011 3:58 PM To: Hyoyoung Chang Cc: Daniel Juyung Seo; enlightenment-devel@lists.**sourceforge.net Subject: Re: [E-devel] [Patch][Evil] evil_string has strcasestr Dear All, Hi~ Here is the patch added on the elementary. :-] Sincerely, Shinwoo Kim. 2011/6/22 Hyoyoung Chang As I know strcasestr is not standard function. It's a glibc extension. CONFORMING TO The strstr() function conforms to C89 and C99. The strcasestr() funcÿÿ tion is a nonstandard extension. -- man page of strcasestr Is it only used in elementary? If then how about add to check routine in configure and add in elm_util.c? Or is it widely used in e project, it can be added in other module. PS. I google about strcasestr. It's contained in LSB 3.1 http://refspecs.freestandards.**org/LSB_3.1.1/LSB-Core-** generic/LSB-Core-generi<http://refspecs.freestandards.org/LSB_3.1.1/LSB-Core-generic/LSB-Core-generi> c/baselib-strcasestr.html -Original Message- From: cnook [mailto:kimci...@gmail.com] Sent: Wednesday, June 22, 2011 5:56 PM To: Daniel Juyung Seo Cc: enlightenment-devel@lists.**sourceforge.net Subject: Re: [E-devel] [Patch][Evil] evil_string has strcasestr Dear Mr. Daniel, Thanks your response :-] Initially, I had added this to the elm_entry in downstream(?), but a reviewer does not prefer it in the elm_entry. Then how about add this somewhere in Elementary? um... btw... why I added this to Evil module is.. there is no strcasestr on MinGW Env. only. Sincerely, Shinwoo Kim. 2011/6/22 Daniel Juyung Seo Well.. Dear Shinwoo Kim, According to vtorri's comment, you can merge this patch and another patch that uses this feature. Thanks. Daniel Juyung Seo (SeoZ) On Wed, Jun 22, 2011 at 4:53 PM, Vincent Torri wrote: On Wed, 22 Jun 2011, cnook wrote: Thanks for your response. There is a possibility in newly made elementary widget. So far, no one have used strcasestr in upstream. i won't add functions if they are not needed. Evil is minimalist and must remain like that. If strcasestr is needed, it will be added. Vincent Sincerely, Shinwoo Kim. 2011/6/22 Vincent Torri Hey There is no strcasestr() on MinGW Env. So this patch comes.. Please review the attached one. where is strcasestr used ? Vincent --**--** -- Simplify data backup and recovery for your virtual environment with vRanger. Installation's a snap, and flexible recovery options mean your data is safe, secure and there when you need it. Data protection magic? Nope - It's vRanger. Get your free trial download today. http://p.sf.net/sfu/quest-**sfdev2dev<http://p.sf.net/sfu/quest-sfdev2dev> __**_ enlightenment-devel mailing list enlightenment-devel@lists.**sourceforge.net https://lists.sourceforge.net/**lists/listinfo/enlightenment-**devel<https://lists.sourceforge.net/lists/listinfo/enlightenment-devel> --**--** -- Simplify data backup and recovery for your virtual environment with vRanger. Installation's a snap, and flexible recovery options mean your data is safe, secure and there when you need it. Data protection magic? Nope - It's vR
Re: [E-devel] [Patch][Evil] evil_string has strcasestr
Oh I see, I got the point. Thanks for your response. Wherever the strcasestr() is used using previous patch, would be changed as bellow. #ifdef HAVE_STRCASESTR strcase(h, n); #else _elm_util_strcasestr(h, n); #endif Anyhow, I have refined the previous patch and attached. Then, wherever the strcasestr() is used, should be changed just as bellow _elm_util_strcasestr(h, n); Sincerely, Shinwoo Kim. 2011/6/29 Vincent Torri > > what he means is: > > * using AC_CHECK_FUNCS([strcasestr]) in configure.ac defined > HAVE_STRCASESTR in config.h > * in the code, you use the macro HAVE_STRCASESTR to use the glibc version > of strcase, or to use your own version > > Vincent > > > On Wed, 29 Jun 2011, cnook wrote: > > Hi >> >> Thanks for your response. >> I have no enough idea when it comes to E16. >> Actually this patch comes because of the Evil Windows environments - >> strcasestr does not exist. >> If strcasestr alternative - Estrcasestr - is on the E16, Should I install >> the E16 also or not? >> >> Sincerely, >> Shinwoo Kim. >> >> >> 2011/6/29 Hyoyoung Chang >> >> I think it's a nice patch if glibc's strcasestr can be used >>> >>> How about to add statements like below? >>> >>> #ifdef HAVE_STRCASESTR >>> #define Estrcasestr(haystack, needle) strcasestr(haystack, needle) >>> #else >>> const char *Estrcasestr(const char *haystack, const char >>> *needle); >>> #endif >>> >>> In E16/e/src/util.h >>> >>> >>> >>> From: cnook [mailto:kimci...@gmail.com] >>>> Sent: Tuesday, June 28, 2011 3:58 PM >>>> To: Hyoyoung Chang >>>> Cc: Daniel Juyung Seo; >>>> enlightenment-devel@lists.**sourceforge.net >>>> Subject: Re: [E-devel] [Patch][Evil] evil_string has strcasestr >>>> >>>> Dear All, >>>> >>>> Hi~ >>>> Here is the patch added on the elementary. :-] >>>> >>>> Sincerely, >>>> Shinwoo Kim. >>>> 2011/6/22 Hyoyoung Chang >>>> As I know strcasestr is not standard function. >>>> It's a glibc extension. >>>> >>>> CONFORMING TO >>>> The strstr() function conforms to C89 and C99. The strcasestr() >>>> funcÿÿ >>>> >>>> tion is a nonstandard extension. >>>> -- man page of strcasestr >>>> >>>> Is it only used in elementary? If then how about add to check routine in >>>> configure and add in elm_util.c? >>>> Or is it widely used in e project, it can be added in other module. >>>> >>>> PS. I google about strcasestr. It's contained in LSB 3.1 >>>> >>>> >>>> http://refspecs.freestandards.**org/LSB_3.1.1/LSB-Core-** >>> generic/LSB-Core-generi<http://refspecs.freestandards.org/LSB_3.1.1/LSB-Core-generic/LSB-Core-generi> >>> >>>> c/baselib-strcasestr.html >>>> >>>> >>>> -Original Message- >>>>> From: cnook [mailto:kimci...@gmail.com] >>>>> Sent: Wednesday, June 22, 2011 5:56 PM >>>>> To: Daniel Juyung Seo >>>>> Cc: >>>>> enlightenment-devel@lists.**sourceforge.net >>>>> Subject: Re: [E-devel] [Patch][Evil] evil_string has strcasestr >>>>> >>>>> Dear Mr. Daniel, >>>>> >>>>> Thanks your response :-] >>>>> Initially, I had added this to the elm_entry in downstream(?), but a >>>>> reviewer does not prefer it in the elm_entry. >>>>> Then how about add this somewhere in Elementary? >>>>> um... btw... why I added this to Evil module is.. there is no >>>>> >>>> strcasestr >>> >>>> on >>>>> MinGW Env. only. >>>>> >>>>> >>>>> Sincerely, >>>>> Shinwoo Kim. >>>>> >>>>> >>>>> 2011/6/22 Daniel Juyung Seo >>>>> >>>>> Well.. >>>>>> Dear Shinwoo Kim, >>>>>> According to vtorri's comment, you can merge this patch and another >>>>>> patch that uses this feature. >>>>>> >>>>>> Thanks. >>>>>> >>>>>> Daniel Juyung Seo (SeoZ) >>>>>> >>>>>> On Wed, Jun 22, 2011 at 4:53 PM, Vincent Torri >>>>>> wrote: >
Re: [E-devel] [Patch][Evil] evil_string has strcasestr
what he means is: * using AC_CHECK_FUNCS([strcasestr]) in configure.ac defined HAVE_STRCASESTR in config.h * in the code, you use the macro HAVE_STRCASESTR to use the glibc version of strcase, or to use your own version Vincent On Wed, 29 Jun 2011, cnook wrote: Hi Thanks for your response. I have no enough idea when it comes to E16. Actually this patch comes because of the Evil Windows environments - strcasestr does not exist. If strcasestr alternative - Estrcasestr - is on the E16, Should I install the E16 also or not? Sincerely, Shinwoo Kim. 2011/6/29 Hyoyoung Chang I think it's a nice patch if glibc's strcasestr can be used How about to add statements like below? #ifdef HAVE_STRCASESTR #define Estrcasestr(haystack, needle) strcasestr(haystack, needle) #else const char *Estrcasestr(const char *haystack, const char *needle); #endif In E16/e/src/util.h From: cnook [mailto:kimci...@gmail.com] Sent: Tuesday, June 28, 2011 3:58 PM To: Hyoyoung Chang Cc: Daniel Juyung Seo; enlightenment-devel@lists.sourceforge.net Subject: Re: [E-devel] [Patch][Evil] evil_string has strcasestr Dear All, Hi~ Here is the patch added on the elementary. :-] Sincerely, Shinwoo Kim. 2011/6/22 Hyoyoung Chang As I know strcasestr is not standard function. It's a glibc extension. CONFORMING TO The strstr() function conforms to C89 and C99. The strcasestr() funcÿÿ tion is a nonstandard extension. -- man page of strcasestr Is it only used in elementary? If then how about add to check routine in configure and add in elm_util.c? Or is it widely used in e project, it can be added in other module. PS. I google about strcasestr. It's contained in LSB 3.1 http://refspecs.freestandards.org/LSB_3.1.1/LSB-Core-generic/LSB-Core-generi c/baselib-strcasestr.html -Original Message- From: cnook [mailto:kimci...@gmail.com] Sent: Wednesday, June 22, 2011 5:56 PM To: Daniel Juyung Seo Cc: enlightenment-devel@lists.sourceforge.net Subject: Re: [E-devel] [Patch][Evil] evil_string has strcasestr Dear Mr. Daniel, Thanks your response :-] Initially, I had added this to the elm_entry in downstream(?), but a reviewer does not prefer it in the elm_entry. Then how about add this somewhere in Elementary? um... btw... why I added this to Evil module is.. there is no strcasestr on MinGW Env. only. Sincerely, Shinwoo Kim. 2011/6/22 Daniel Juyung Seo Well.. Dear Shinwoo Kim, According to vtorri's comment, you can merge this patch and another patch that uses this feature. Thanks. Daniel Juyung Seo (SeoZ) On Wed, Jun 22, 2011 at 4:53 PM, Vincent Torri wrote: On Wed, 22 Jun 2011, cnook wrote: Thanks for your response. There is a possibility in newly made elementary widget. So far, no one have used strcasestr in upstream. i won't add functions if they are not needed. Evil is minimalist and must remain like that. If strcasestr is needed, it will be added. Vincent Sincerely, Shinwoo Kim. 2011/6/22 Vincent Torri Hey There is no strcasestr() on MinGW Env. So this patch comes.. Please review the attached one. where is strcasestr used ? Vincent -- Simplify data backup and recovery for your virtual environment with vRanger. Installation's a snap, and flexible recovery options mean your data is safe, secure and there when you need it. Data protection magic? Nope - It's vRanger. Get your free trial download today. http://p.sf.net/sfu/quest-sfdev2dev ___ enlightenment-devel mailing list enlightenment-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-devel -- Simplify data backup and recovery for your virtual environment with vRanger. Installation's a snap, and flexible recovery options mean your data is safe, secure and there when you need it. Data protection magic? Nope - It's vRanger. Get your free trial download today. http://p.sf.net/sfu/quest-sfdev2dev ___ enlightenment-devel mailing list enlightenment-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-devel -- All of the data generated in your IT infrastructure is seriously valuable. Why? It contains a definitive record of application performance, security threats, fraudulent activity, and more. Splunk takes this data and makes sense of it. IT sense. And common sense. http://p.sf.net/sfu/splunk-d2d-c2 ___ enlightenment-devel mailing list enlightenment-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-devel -
Re: [E-devel] [Patch][Evil] evil_string has strcasestr
Hi Thanks for your response. I have no enough idea when it comes to E16. Actually this patch comes because of the Evil Windows environments - strcasestr does not exist. If strcasestr alternative - Estrcasestr - is on the E16, Should I install the E16 also or not? Sincerely, Shinwoo Kim. 2011/6/29 Hyoyoung Chang > I think it's a nice patch if glibc's strcasestr can be used > > How about to add statements like below? > > #ifdef HAVE_STRCASESTR > #define Estrcasestr(haystack, needle) strcasestr(haystack, needle) > #else > const char *Estrcasestr(const char *haystack, const char *needle); > #endif > > In E16/e/src/util.h > > > > > From: cnook [mailto:kimci...@gmail.com] > > Sent: Tuesday, June 28, 2011 3:58 PM > > To: Hyoyoung Chang > > Cc: Daniel Juyung Seo; enlightenment-devel@lists.sourceforge.net > > Subject: Re: [E-devel] [Patch][Evil] evil_string has strcasestr > > > > Dear All, > > > > Hi~ > > Here is the patch added on the elementary. :-] > > > > Sincerely, > > Shinwoo Kim. > > 2011/6/22 Hyoyoung Chang > > As I know strcasestr is not standard function. > > It's a glibc extension. > > > > CONFORMING TO > > The strstr() function conforms to C89 and C99. The strcasestr() > > func‐ > > tion is a nonstandard extension. > > -- man page of strcasestr > > > > Is it only used in elementary? If then how about add to check routine in > > configure and add in elm_util.c? > > Or is it widely used in e project, it can be added in other module. > > > > PS. I google about strcasestr. It's contained in LSB 3.1 > > > > > http://refspecs.freestandards.org/LSB_3.1.1/LSB-Core-generic/LSB-Core-generi > > c/baselib-strcasestr.html > > > > > > > -Original Message- > > > From: cnook [mailto:kimci...@gmail.com] > > > Sent: Wednesday, June 22, 2011 5:56 PM > > > To: Daniel Juyung Seo > > > Cc: enlightenment-devel@lists.sourceforge.net > > > Subject: Re: [E-devel] [Patch][Evil] evil_string has strcasestr > > > > > > Dear Mr. Daniel, > > > > > > Thanks your response :-] > > > Initially, I had added this to the elm_entry in downstream(?), but a > > > reviewer does not prefer it in the elm_entry. > > > Then how about add this somewhere in Elementary? > > > um... btw... why I added this to Evil module is.. there is no > strcasestr > > > on > > > MinGW Env. only. > > > > > > > > > Sincerely, > > > Shinwoo Kim. > > > > > > > > > 2011/6/22 Daniel Juyung Seo > > > > > > > Well.. > > > > Dear Shinwoo Kim, > > > > According to vtorri's comment, you can merge this patch and another > > > > patch that uses this feature. > > > > > > > > Thanks. > > > > > > > > Daniel Juyung Seo (SeoZ) > > > > > > > > On Wed, Jun 22, 2011 at 4:53 PM, Vincent Torri > > > > wrote: > > > > > > > > > > > > > > > On Wed, 22 Jun 2011, cnook wrote: > > > > > > > > > >> Thanks for your response. > > > > >> There is a possibility in newly made elementary widget. > > > > >> So far, no one have used strcasestr in upstream. > > > > > > > > > > i won't add functions if they are not needed. Evil is minimalist > and > > > must > > > > > remain like that. If strcasestr is needed, it will be added. > > > > > > > > > > Vincent > > > > > > > > > >> > > > > >> Sincerely, > > > > >> Shinwoo Kim. > > > > >> > > > > >> 2011/6/22 Vincent Torri > > > > >> > > > > >>> > > > > >>> Hey > > > > >>> > > > > >>> > > > > >>> There is no strcasestr() on MinGW Env. > > > > >>>> So this patch comes.. Please review the attached one. > > > > >>>> > > > > >>> > > > > >>> where is strcasestr used ? > > > > >>> > > > > >>> Vincent > > > > >>> > > > > >> > > > > > > > > > > > > > > > > > >
Re: [E-devel] [Patch][Evil] evil_string has strcasestr
I think it's a nice patch if glibc's strcasestr can be used How about to add statements like below? #ifdef HAVE_STRCASESTR #define Estrcasestr(haystack, needle) strcasestr(haystack, needle) #else const char *Estrcasestr(const char *haystack, const char *needle); #endif In E16/e/src/util.h > From: cnook [mailto:kimci...@gmail.com] > Sent: Tuesday, June 28, 2011 3:58 PM > To: Hyoyoung Chang > Cc: Daniel Juyung Seo; enlightenment-devel@lists.sourceforge.net > Subject: Re: [E-devel] [Patch][Evil] evil_string has strcasestr > > Dear All, > > Hi~ > Here is the patch added on the elementary. :-] > > Sincerely, > Shinwoo Kim. > 2011/6/22 Hyoyoung Chang > As I know strcasestr is not standard function. > It's a glibc extension. > > CONFORMING TO > The strstr() function conforms to C89 and C99. The strcasestr() > func‐ > tion is a nonstandard extension. > -- man page of strcasestr > > Is it only used in elementary? If then how about add to check routine in > configure and add in elm_util.c? > Or is it widely used in e project, it can be added in other module. > > PS. I google about strcasestr. It's contained in LSB 3.1 > > http://refspecs.freestandards.org/LSB_3.1.1/LSB-Core-generic/LSB-Core-generi > c/baselib-strcasestr.html > > > > -Original Message- > > From: cnook [mailto:kimci...@gmail.com] > > Sent: Wednesday, June 22, 2011 5:56 PM > > To: Daniel Juyung Seo > > Cc: enlightenment-devel@lists.sourceforge.net > > Subject: Re: [E-devel] [Patch][Evil] evil_string has strcasestr > > > > Dear Mr. Daniel, > > > > Thanks your response :-] > > Initially, I had added this to the elm_entry in downstream(?), but a > > reviewer does not prefer it in the elm_entry. > > Then how about add this somewhere in Elementary? > > um... btw... why I added this to Evil module is.. there is no strcasestr > > on > > MinGW Env. only. > > > > > > Sincerely, > > Shinwoo Kim. > > > > > > 2011/6/22 Daniel Juyung Seo > > > > > Well.. > > > Dear Shinwoo Kim, > > > According to vtorri's comment, you can merge this patch and another > > > patch that uses this feature. > > > > > > Thanks. > > > > > > Daniel Juyung Seo (SeoZ) > > > > > > On Wed, Jun 22, 2011 at 4:53 PM, Vincent Torri > > > wrote: > > > > > > > > > > > > On Wed, 22 Jun 2011, cnook wrote: > > > > > > > >> Thanks for your response. > > > >> There is a possibility in newly made elementary widget. > > > >> So far, no one have used strcasestr in upstream. > > > > > > > > i won't add functions if they are not needed. Evil is minimalist and > > must > > > > remain like that. If strcasestr is needed, it will be added. > > > > > > > > Vincent > > > > > > > >> > > > >> Sincerely, > > > >> Shinwoo Kim. > > > >> > > > >> 2011/6/22 Vincent Torri > > > >> > > > >>> > > > >>> Hey > > > >>> > > > >>> > > > >>> There is no strcasestr() on MinGW Env. > > > >>>> So this patch comes.. Please review the attached one. > > > >>>> > > > >>> > > > >>> where is strcasestr used ? > > > >>> > > > >>> Vincent > > > >>> > > > >> > > > > > > > > > > > > > -- > > > > Simplify data backup and recovery for your virtual environment with > > > vRanger. > > > > Installation's a snap, and flexible recovery options mean your data is > > > safe, > > > > secure and there when you need it. Data protection magic? > > > > Nope - It's vRanger. Get your free trial download today. > > > > http://p.sf.net/sfu/quest-sfdev2dev > > > > ___ > > > > enlightenment-devel mailing list > > > > enlightenment-devel@lists.sourceforge.net > > > > https://lists.sourceforge.net/lists/listinfo/enlightenment-devel > > > > > > > > > -- > > > > Simplify data backup and recovery for your virtual environment with > > vRanger. > > Installation's a snap, and flexible recovery options mean your data is > > safe, > > secure and there when you need it. Data protection magic? > > Nope - It's vRanger. Get your free trial download today. > > http://p.sf.net/sfu/quest-sfdev2dev > > ___ > > enlightenment-devel mailing list > > enlightenment-devel@lists.sourceforge.net > > https://lists.sourceforge.net/lists/listinfo/enlightenment-devel -- All of the data generated in your IT infrastructure is seriously valuable. Why? It contains a definitive record of application performance, security threats, fraudulent activity, and more. Splunk takes this data and makes sense of it. IT sense. And common sense. http://p.sf.net/sfu/splunk-d2d-c2 ___ enlightenment-devel mailing list enlightenment-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
Re: [E-devel] [Patch][Evil] evil_string has strcasestr
Dear All, Hi~ Here is the patch added on the elementary. :-] Sincerely, Shinwoo Kim. 2011/6/22 Hyoyoung Chang > As I know strcasestr is not standard function. > It's a glibc extension. > > CONFORMING TO > The strstr() function conforms to C89 and C99. The strcasestr() > func‐ > tion is a nonstandard extension. > -- man page of strcasestr > > Is it only used in elementary? If then how about add to check routine in > configure and add in elm_util.c? > Or is it widely used in e project, it can be added in other module. > > PS. I google about strcasestr. It's contained in LSB 3.1 > > > http://refspecs.freestandards.org/LSB_3.1.1/LSB-Core-generic/LSB-Core-generi > c/baselib-strcasestr.html > > > > -Original Message- > > From: cnook [mailto:kimci...@gmail.com] > > Sent: Wednesday, June 22, 2011 5:56 PM > > To: Daniel Juyung Seo > > Cc: enlightenment-devel@lists.sourceforge.net > > Subject: Re: [E-devel] [Patch][Evil] evil_string has strcasestr > > > > Dear Mr. Daniel, > > > > Thanks your response :-] > > Initially, I had added this to the elm_entry in downstream(?), but a > > reviewer does not prefer it in the elm_entry. > > Then how about add this somewhere in Elementary? > > um... btw... why I added this to Evil module is.. there is no strcasestr > > on > > MinGW Env. only. > > > > > > Sincerely, > > Shinwoo Kim. > > > > > > 2011/6/22 Daniel Juyung Seo > > > > > Well.. > > > Dear Shinwoo Kim, > > > According to vtorri's comment, you can merge this patch and another > > > patch that uses this feature. > > > > > > Thanks. > > > > > > Daniel Juyung Seo (SeoZ) > > > > > > On Wed, Jun 22, 2011 at 4:53 PM, Vincent Torri > > > wrote: > > > > > > > > > > > > On Wed, 22 Jun 2011, cnook wrote: > > > > > > > >> Thanks for your response. > > > >> There is a possibility in newly made elementary widget. > > > >> So far, no one have used strcasestr in upstream. > > > > > > > > i won't add functions if they are not needed. Evil is minimalist and > > must > > > > remain like that. If strcasestr is needed, it will be added. > > > > > > > > Vincent > > > > > > > >> > > > >> Sincerely, > > > >> Shinwoo Kim. > > > >> > > > >> 2011/6/22 Vincent Torri > > > >> > > > >>> > > > >>> Hey > > > >>> > > > >>> > > > >>> There is no strcasestr() on MinGW Env. > > > >>>> So this patch comes.. Please review the attached one. > > > >>>> > > > >>> > > > >>> where is strcasestr used ? > > > >>> > > > >>> Vincent > > > >>> > > > >> > > > > > > > > > > > > > > -- > > > > Simplify data backup and recovery for your virtual environment with > > > vRanger. > > > > Installation's a snap, and flexible recovery options mean your data > is > > > safe, > > > > secure and there when you need it. Data protection magic? > > > > Nope - It's vRanger. Get your free trial download today. > > > > http://p.sf.net/sfu/quest-sfdev2dev > > > > ___ > > > > enlightenment-devel mailing list > > > > enlightenment-devel@lists.sourceforge.net > > > > https://lists.sourceforge.net/lists/listinfo/enlightenment-devel > > > > > > > > > > -- > > > > Simplify data backup and recovery for your virtual environment with > > vRanger. > > Installation's a snap, and flexible recovery options mean your data is > > safe, > > secure and there when you need it. Data protection magic? > > Nope - It's vRanger. Get your free trial download today. > > http://p.sf.net/sfu/quest-sfdev2dev > > ___ > > enlightenment-devel mailing list > > enlightenment-devel@lists.sourceforge.net > > https://lists.sourceforge.net/lists/listinfo/enlightenment-devel > > Index:
Re: [E-devel] [Patch][Evil] evil_string has strcasestr
As I know strcasestr is not standard function. It's a glibc extension. CONFORMING TO The strstr() function conforms to C89 and C99. The strcasestr() func‐ tion is a nonstandard extension. -- man page of strcasestr Is it only used in elementary? If then how about add to check routine in configure and add in elm_util.c? Or is it widely used in e project, it can be added in other module. PS. I google about strcasestr. It's contained in LSB 3.1 http://refspecs.freestandards.org/LSB_3.1.1/LSB-Core-generic/LSB-Core-generi c/baselib-strcasestr.html > -Original Message- > From: cnook [mailto:kimci...@gmail.com] > Sent: Wednesday, June 22, 2011 5:56 PM > To: Daniel Juyung Seo > Cc: enlightenment-devel@lists.sourceforge.net > Subject: Re: [E-devel] [Patch][Evil] evil_string has strcasestr > > Dear Mr. Daniel, > > Thanks your response :-] > Initially, I had added this to the elm_entry in downstream(?), but a > reviewer does not prefer it in the elm_entry. > Then how about add this somewhere in Elementary? > um... btw... why I added this to Evil module is.. there is no strcasestr > on > MinGW Env. only. > > > Sincerely, > Shinwoo Kim. > > > 2011/6/22 Daniel Juyung Seo > > > Well.. > > Dear Shinwoo Kim, > > According to vtorri's comment, you can merge this patch and another > > patch that uses this feature. > > > > Thanks. > > > > Daniel Juyung Seo (SeoZ) > > > > On Wed, Jun 22, 2011 at 4:53 PM, Vincent Torri > > wrote: > > > > > > > > > On Wed, 22 Jun 2011, cnook wrote: > > > > > >> Thanks for your response. > > >> There is a possibility in newly made elementary widget. > > >> So far, no one have used strcasestr in upstream. > > > > > > i won't add functions if they are not needed. Evil is minimalist and > must > > > remain like that. If strcasestr is needed, it will be added. > > > > > > Vincent > > > > > >> > > >> Sincerely, > > >> Shinwoo Kim. > > >> > > >> 2011/6/22 Vincent Torri > > >> > > >>> > > >>> Hey > > >>> > > >>> > > >>> There is no strcasestr() on MinGW Env. > > >>>> So this patch comes.. Please review the attached one. > > >>>> > > >>> > > >>> where is strcasestr used ? > > >>> > > >>> Vincent > > >>> > > >> > > > > > > > > > -- > > > Simplify data backup and recovery for your virtual environment with > > vRanger. > > > Installation's a snap, and flexible recovery options mean your data is > > safe, > > > secure and there when you need it. Data protection magic? > > > Nope - It's vRanger. Get your free trial download today. > > > http://p.sf.net/sfu/quest-sfdev2dev > > > ___ > > > enlightenment-devel mailing list > > > enlightenment-devel@lists.sourceforge.net > > > https://lists.sourceforge.net/lists/listinfo/enlightenment-devel > > > > > > -- > > Simplify data backup and recovery for your virtual environment with > vRanger. > Installation's a snap, and flexible recovery options mean your data is > safe, > secure and there when you need it. Data protection magic? > Nope - It's vRanger. Get your free trial download today. > http://p.sf.net/sfu/quest-sfdev2dev > ___ > enlightenment-devel mailing list > enlightenment-devel@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/enlightenment-devel -- Simplify data backup and recovery for your virtual environment with vRanger. Installation's a snap, and flexible recovery options mean your data is safe, secure and there when you need it. Data protection magic? Nope - It's vRanger. Get your free trial download today. http://p.sf.net/sfu/quest-sfdev2dev ___ enlightenment-devel mailing list enlightenment-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
Re: [E-devel] [Patch][Evil] evil_string has strcasestr
Dear Mr. Daniel, Thanks your response :-] Initially, I had added this to the elm_entry in downstream(?), but a reviewer does not prefer it in the elm_entry. Then how about add this somewhere in Elementary? um... btw... why I added this to Evil module is.. there is no strcasestr on MinGW Env. only. Sincerely, Shinwoo Kim. 2011/6/22 Daniel Juyung Seo > Well.. > Dear Shinwoo Kim, > According to vtorri's comment, you can merge this patch and another > patch that uses this feature. > > Thanks. > > Daniel Juyung Seo (SeoZ) > > On Wed, Jun 22, 2011 at 4:53 PM, Vincent Torri > wrote: > > > > > > On Wed, 22 Jun 2011, cnook wrote: > > > >> Thanks for your response. > >> There is a possibility in newly made elementary widget. > >> So far, no one have used strcasestr in upstream. > > > > i won't add functions if they are not needed. Evil is minimalist and must > > remain like that. If strcasestr is needed, it will be added. > > > > Vincent > > > >> > >> Sincerely, > >> Shinwoo Kim. > >> > >> 2011/6/22 Vincent Torri > >> > >>> > >>> Hey > >>> > >>> > >>> There is no strcasestr() on MinGW Env. > So this patch comes.. Please review the attached one. > > >>> > >>> where is strcasestr used ? > >>> > >>> Vincent > >>> > >> > > > > > -- > > Simplify data backup and recovery for your virtual environment with > vRanger. > > Installation's a snap, and flexible recovery options mean your data is > safe, > > secure and there when you need it. Data protection magic? > > Nope - It's vRanger. Get your free trial download today. > > http://p.sf.net/sfu/quest-sfdev2dev > > ___ > > enlightenment-devel mailing list > > enlightenment-devel@lists.sourceforge.net > > https://lists.sourceforge.net/lists/listinfo/enlightenment-devel > > > -- Simplify data backup and recovery for your virtual environment with vRanger. Installation's a snap, and flexible recovery options mean your data is safe, secure and there when you need it. Data protection magic? Nope - It's vRanger. Get your free trial download today. http://p.sf.net/sfu/quest-sfdev2dev ___ enlightenment-devel mailing list enlightenment-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
Re: [E-devel] [Patch][Evil] evil_string has strcasestr
Well.. Dear Shinwoo Kim, According to vtorri's comment, you can merge this patch and another patch that uses this feature. Thanks. Daniel Juyung Seo (SeoZ) On Wed, Jun 22, 2011 at 4:53 PM, Vincent Torri wrote: > > > On Wed, 22 Jun 2011, cnook wrote: > >> Thanks for your response. >> There is a possibility in newly made elementary widget. >> So far, no one have used strcasestr in upstream. > > i won't add functions if they are not needed. Evil is minimalist and must > remain like that. If strcasestr is needed, it will be added. > > Vincent > >> >> Sincerely, >> Shinwoo Kim. >> >> 2011/6/22 Vincent Torri >> >>> >>> Hey >>> >>> >>> There is no strcasestr() on MinGW Env. So this patch comes.. Please review the attached one. >>> >>> where is strcasestr used ? >>> >>> Vincent >>> >> > > -- > Simplify data backup and recovery for your virtual environment with vRanger. > Installation's a snap, and flexible recovery options mean your data is safe, > secure and there when you need it. Data protection magic? > Nope - It's vRanger. Get your free trial download today. > http://p.sf.net/sfu/quest-sfdev2dev > ___ > enlightenment-devel mailing list > enlightenment-devel@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/enlightenment-devel > -- Simplify data backup and recovery for your virtual environment with vRanger. Installation's a snap, and flexible recovery options mean your data is safe, secure and there when you need it. Data protection magic? Nope - It's vRanger. Get your free trial download today. http://p.sf.net/sfu/quest-sfdev2dev ___ enlightenment-devel mailing list enlightenment-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
Re: [E-devel] [Patch][Evil] evil_string has strcasestr
On Wed, 22 Jun 2011, cnook wrote: > Thanks for your response. > There is a possibility in newly made elementary widget. > So far, no one have used strcasestr in upstream. i won't add functions if they are not needed. Evil is minimalist and must remain like that. If strcasestr is needed, it will be added. Vincent > > Sincerely, > Shinwoo Kim. > > 2011/6/22 Vincent Torri > >> >> Hey >> >> >> There is no strcasestr() on MinGW Env. >>> So this patch comes.. Please review the attached one. >>> >> >> where is strcasestr used ? >> >> Vincent >> > -- Simplify data backup and recovery for your virtual environment with vRanger. Installation's a snap, and flexible recovery options mean your data is safe, secure and there when you need it. Data protection magic? Nope - It's vRanger. Get your free trial download today. http://p.sf.net/sfu/quest-sfdev2dev ___ enlightenment-devel mailing list enlightenment-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
Re: [E-devel] [Patch][Evil] evil_string has strcasestr
Thanks for your response. There is a possibility in newly made elementary widget. So far, no one have used strcasestr in upstream. Sincerely, Shinwoo Kim. 2011/6/22 Vincent Torri > > Hey > > > There is no strcasestr() on MinGW Env. >> So this patch comes.. Please review the attached one. >> > > where is strcasestr used ? > > Vincent > -- Simplify data backup and recovery for your virtual environment with vRanger. Installation's a snap, and flexible recovery options mean your data is safe, secure and there when you need it. Data protection magic? Nope - It's vRanger. Get your free trial download today. http://p.sf.net/sfu/quest-sfdev2dev ___ enlightenment-devel mailing list enlightenment-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
Re: [E-devel] [Patch][Evil] evil_string has strcasestr
Hey > There is no strcasestr() on MinGW Env. > So this patch comes.. Please review the attached one. where is strcasestr used ? Vincent -- Simplify data backup and recovery for your virtual environment with vRanger. Installation's a snap, and flexible recovery options mean your data is safe, secure and there when you need it. Data protection magic? Nope - It's vRanger. Get your free trial download today. http://p.sf.net/sfu/quest-sfdev2dev ___ enlightenment-devel mailing list enlightenment-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
[E-devel] [Patch][Evil] evil_string has strcasestr
Hi All,
There is no strcasestr() on MinGW Env.
So this patch comes.. Please review the attached one.
Sincerely,
Shinwoo Kim.
Index: src/lib/evil_string.c
===
--- src/lib/evil_string.c (revision 60564)
+++ src/lib/evil_string.c (working copy)
@@ -3,8 +3,8 @@
#endif /* HAVE_CONFIG_H */
#include "Evil.h"
+#include
-
#ifdef _WIN32_WCE
/*
@@ -84,6 +84,31 @@
return ret;
}
+char *
+strcasestr (char *h, char *n)
+{
+ char *a, *e;
+ a = h;
+ e = n;
+
+ if (!h || !*h || !n ||!*n) return 0;
+ while (*a && *e)
+ {
+if(toupper((unsigned char) *a) != toupper((unsigned char) *e))
+ {
+ ++h;
+ a = h;
+ e = n;
+ }
+else
+ {
+ ++a;
+ ++e;
+ }
+ }
+ return *e ? 0 : h;
+}
+
#ifdef _MSC_VER
int strcasecmp(const char *s1, const char *s2)
Index: src/lib/evil_string.h
===
--- src/lib/evil_string.h (revision 60564)
+++ src/lib/evil_string.h (working copy)
@@ -103,6 +103,11 @@
*/
EAPI char *strrstr (const char *str, const char *substr);
+/**
+ * @brief Similar to 'strstr' but this function ignores the case of both strings.
+ */
+EAPI char *strcasestr (char *str, char *substr);
+
#ifdef _MSC_VER
EAPI int strcasecmp(const char *s1, const char *s2);
--
Simplify data backup and recovery for your virtual environment with vRanger.
Installation's a snap, and flexible recovery options mean your data is safe,
secure and there when you need it. Data protection magic?
Nope - It's vRanger. Get your free trial download today.
http://p.sf.net/sfu/quest-sfdev2dev___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
