On Fri, 8 Jul 2011 21:27:21 +0900 cnook <kimci...@gmail.com> 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 <ras...@rasterman.com> > > > On Fri, 1 Jul 2011 10:15:21 +0900 cnook <kimci...@gmail.com> 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 <vto...@univ-evry.fr> > > > > > > > > > > > > > > > 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 <vto...@univ-evry.fr> > > > >> > > > >> > > > >>> 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 <hyoyoung.ch...@samsung.com> > > > >>>> > > > >>>> 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> > > > >>>>>> <enlightenment-**de...@lists.sourceforge.net< > > 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 <hyoyoung.ch...@samsung.com> > > > >>>>>> 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< > > 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.**so**urceforge.net< > > http://sourceforge.net> > > > >>>>>>> <enlightenment-**de...@lists.sourceforge.net< > > 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 <seojuyu...@gmail.com> > > > >>>>>>> > > > >>>>>>> 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 < > > vto...@univ-evry.fr > > > >>>>>>>> > > > > >>>>>>>> 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 <vto...@univ-evry.fr> > > > >>>>>>>>>> > > > >>>>>>>>>> > > > >>>>>>>>>> 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> > > > >>>>>>>>> <http://p.sf.net/sfu/**quest-sfdev2dev< > > http://p.sf.net/sfu/quest-sfdev2dev> > > > >>>>>>>>> > > > > >>>>>>>>> ______________________________****_________________ > > > >>>>>>>>> enlightenment-devel mailing list > > > >>>>>>>>> enlightenment-devel@lists.**so**urceforge.net< > > http://sourceforge.net> > > > >>>>>>>>> <enlightenment-**de...@lists.sourceforge.net< > > enlightenment-devel@lists.sourceforge.net> > > > >>>>>>>>> > > > > >>>>>>>>> https://lists.sourceforge.net/****lists/listinfo/** > > > >>>>>>>>> enlightenment-**devel< > > https://lists.sourceforge.net/**lists/listinfo/enlightenment-**devel> > > > >>>>>>>>> <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 vRanger. Get your free trial download today. > > > >>>>>>> http://p.sf.net/sfu/quest-****sfdev2dev< > > http://p.sf.net/sfu/quest-**sfdev2dev> > > > >>>>>>> <http://p.sf.net/sfu/**quest-sfdev2dev< > > http://p.sf.net/sfu/quest-sfdev2dev> > > > >>>>>>> > > > > >>>>>>> ______________________________****_________________ > > > >>>>>>> enlightenment-devel mailing list > > > >>>>>>> enlightenment-devel@lists.**so**urceforge.net< > > http://sourceforge.net> > > > >>>>>>> <enlightenment-**de...@lists.sourceforge.net< > > enlightenment-devel@lists.sourceforge.net> > > > >>>>>>> > > > > >>>>>>> https://lists.sourceforge.net/****lists/listinfo/** > > > >>>>>>> enlightenment-**devel< > > https://lists.sourceforge.net/**lists/listinfo/enlightenment-**devel> > > > >>>>>>> <https://**lists.sourceforge.net/lists/** > > > >>>>>>> listinfo/enlightenment-devel< > > 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< > > http://p.sf.net/sfu/splunk-**d2d-c2>< > > > >>>> http://p.sf.net/sfu/splunk-**d2d-c2 < > > http://p.sf.net/sfu/splunk-d2d-c2> > > > >>>> > > > > >>>> > > > >>>> ______________________________****_________________ > > > >>>> enlightenment-devel mailing list > > > >>>> enlightenment-devel@lists.**so**urceforge.net < > > http://sourceforge.net>< > > > >>>> enlightenment-**de...@lists.sourceforge.net< > > enlightenment-devel@lists.sourceforge.net> > > > >>>> > > > > >>>> https://lists.sourceforge.net/****lists/listinfo/** > > > >>>> enlightenment-**devel< > > https://lists.sourceforge.net/**lists/listinfo/enlightenment-**devel> > > > >>>> <https://** > > lists.sourceforge.net/lists/**listinfo/enlightenment-devel< > > https://lists.sourceforge.net/lists/listinfo/enlightenment-devel> > > > >>>> > > > > >>>> > > > >>>> > > > >>>> > > > > > > -- > > ------------- Codito, ergo sum - "I code, therefore I am" -------------- > > The Rasterman (Carsten Haitzler) ras...@rasterman.com > > > > -- ------------- Codito, ergo sum - "I code, therefore I am" -------------- The Rasterman (Carsten Haitzler) ras...@rasterman.com ------------------------------------------------------------------------------ Got Input? Slashdot Needs You. Take our quick survey online. Come on, we don't ask for help often. Plus, you'll get a chance to win $100 to spend on ThinkGeek. http://p.sf.net/sfu/slashdot-survey _______________________________________________ enlightenment-devel mailing list enlightenment-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
