Hello, this is Daniel Juyung Seo.

I have a question about below commit.

It looks like the statement is trying to avoid argc == 2.
Why 'argc > 2 || argc < 2' was used instead of 'argc != 2' ?
Any reasons?

Thank you.
Best Regards,
Daniel Juyung Seo.

On Mon, Aug 23, 2010 at 11:44 PM, Enlightenment SVN
<no-re...@enlightenment.org> wrote:
> Log:
>        * edje: prevent segv when no param is given to edje_convert.
>
> Author:       cedric
> Date:         2010-08-23 07:44:04 -0700 (Mon, 23 Aug 2010)
> New Revision: 51575
>
> Modified:
>  trunk/edje/src/bin/edje_convert_main.c
>
> Modified: trunk/edje/src/bin/edje_convert_main.c
> ===================================================================
> --- trunk/edje/src/bin/edje_convert_main.c      2010-08-23 14:27:50 UTC (rev 
> 51574)
> +++ trunk/edje/src/bin/edje_convert_main.c      2010-08-23 14:44:04 UTC (rev 
> 51575)
> @@ -128,7 +128,7 @@
>    eet_init();
>
>    progname = argv[0];
> -   if (argc > 2 || !strcmp(argv[1], "-h"))
> +   if (argc > 2 || argc < 2 || !strcmp(argv[1], "-h"))
>      {
>        main_help();
>        return 0;
>
>
> ------------------------------------------------------------------------------
> This SF.net email is sponsored by
>
> Make an app they can't live without
> Enter the BlackBerry Developer Challenge
> http://p.sf.net/sfu/RIM-dev2dev
> _______________________________________________
> enlightenment-svn mailing list
> enlightenment-...@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/enlightenment-svn
>

------------------------------------------------------------------------------
Virtualization is moving to the mainstream and overtaking non-virtualized
environment for deploying applications. Does it make network security 
easier or more difficult to achieve? Read this whitepaper to separate the 
two and get a better understanding.
http://p.sf.net/sfu/hp-phase2-d2d
_______________________________________________
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to