[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
FYI: put more `.m4sh' details in HACKING [libtool--devo--1.0--patch-269]
From: |
Gary V. Vaughan |
Subject: |
FYI: put more `.m4sh' details in HACKING [libtool--devo--1.0--patch-269] |
Date: |
Fri, 22 Oct 2004 11:31:09 +0100 (BST) |
User-agent: |
mailnotify/0.5 |
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
Applied to HEAD and branch-2-0.
* looking for address@hidden/libtool--devo--1.0--patch-268 to compare with
* comparing to address@hidden/libtool--devo--1.0--patch-268
M ChangeLog
M HACKING
* modified files
Index: Changelog
from Gary V. Vaughan <address@hidden>
* HACKING: Updated: `.in' files are now `.m4sh'; document some
more coding standards for these files.
--- orig/HACKING
+++ mod/HACKING
@@ -93,12 +93,26 @@
of the loop itself should print each "important" command it runs.
-5. Editing libtoolize.in and ltmain.in
-======================================
+5. Editing `.m4sh' Files
+========================
* Use shell functions, but be careful not to assume local scope for
variable names. Don't use `return', instead echo the result of a
function and call it from within backquotes.
+
+* Function names should be prefixed `func_'.
+
+* For functions that are called frequently, if you need to return a
+ value, don't cause unneccessary forking of the shell using echo as
+ described above; instead set the return value in a variable named
+ after the called function with a suffix of `_result'. For example
+ the function `func_quote_for_eval' stores its return value in the
+ variable `$func_quote_for_eval_result'.
+
+* Although sh-indentation is set to 2 (by the `Local Variables:' block
+ at the end of .m4sh files), the left margin of the body of shell
+ functions should begin indented by 4 spaces.
+
6. Abstraction layers in libltdl
================================
- --
Gary V. Vaughan ())_. address@hidden,gnu.org}
Research Scientist ( '/ http://tkd.kicks-ass.net
GNU Hacker / )= http://www.gnu.org/software/libtool
Technical Author `(_~)_ http://sources.redhat.com/autobook
_________________________________________________________
This patch notification generated by tlaapply version 0.8
http://tkd.kicks-ass.net/arch/address@hidden/cvs-utils--tla--1.0
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.4 (Darwin)
iD8DBQFBeOFsFRMICSmD1gYRAsvMAJ9pX6VnZQzg8g3iAlwBnp3ZGt7X4ACdETIL
Rgo/cCZvPCfqFZbSc/UxHAQ=
=r1Kz
-----END PGP SIGNATURE-----
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- FYI: put more `.m4sh' details in HACKING [libtool--devo--1.0--patch-269],
Gary V. Vaughan <=