On Monday, 07 January 2008, at 21:19:22 (-0500),
Enlightenment CVS wrote:

> -  cfd = e_config_dialog_new (con, D_ ("MPDule Configuration"), "MPDule", 
> "_e_modules_mpdule_config_dialog", buf, 0, v, ci);
> +  cfd =
> +    e_config_dialog_new (con, D_ ("MPDule Configuration"), "MPDule",
> +                      "_e_modules_mpdule_config_dialog", buf, 0, v, ci);

I could not have asked for a better or more clear-cut demonstration of
why the 80-column limit results in less readable code.  :-)

> +  taskbar_config->handlers = evas_list_append
> +    (taskbar_config->handlers, ecore_event_handler_add
> +     (E_EVENT_BORDER_ADD, _taskbar_cb_event_border_add, NULL));

...except maybe that.  Ick.

> Log Message:
> shakes fist at indent!
> 
> ===================================================================
> RCS file: /cvs/e/e_modules/mpdule/mpdule.edc,v
> retrieving revision 1.5
> retrieving revision 1.6
> diff -u -3 -r1.5 -r1.6
> --- mpdule.edc        8 Jan 2008 02:19:22 -0000       1.5
> +++ mpdule.edc        8 Jan 2008 03:05:06 -0000       1.6
      ^^^^^^^^^^
You should never run "indent" on anything but .c and .h files.  It is
not intended to do arbitrary indentation.  For that, you're better off
using M-x indent-region in Emacs.

Michael

-- 
Michael Jennings (a.k.a. KainX)  http://www.kainx.org/  <[EMAIL PROTECTED]>
Linux Server/Cluster Admin, LBL.gov       Author, Eterm (www.eterm.org)
-----------------------------------------------------------------------
 "There is a greater darkness than the one we fight.  It is the 
  darkness of a soul that has lost its way."       -- G'Kar, Babylon 5

-------------------------------------------------------------------------
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
_______________________________________________
enlightenment-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to