Patches

[Jörg Sommer <joerg@xxxxxxxxxxxx>]

Hi John,

what do you think about these patches:

http://git.debian.org/?p=pkg-jed/jed.git;a=commitdiff;h=ee19e705785bcea2fa07e09b029e797152ecdf20
Author: Jörg Sommer <joerg@xxxxxxxxxxxx>
Date:   Sun Aug 24 11:48:33 2008 +0200

    Remove duplicate brace

M       doc/tm/rtl/keys.tm

diff --git a/doc/tm/rtl/keys.tm b/doc/tm/rtl/keys.tm
index f1acea2..9d20d86 100644
--- a/doc/tm/rtl/keys.tm
+++ b/doc/tm/rtl/keys.tm
@@ -328,7 +328,7 @@
      if (arg == NULL)
        message ("No Prefix Argument");
      else
-       vmessage ("Prefix argument: %d", arg));
+       vmessage ("Prefix argument: %d", arg);
 #v-
 \notes
   The old semantics, which are still supported but deprecated allows
@@ -340,7 +340,7 @@
      if (arg == -9999)
        message ("No Prefix Argument");
      else
-       vmessage ("Prefix argument: %d", arg));
+       vmessage ("Prefix argument: %d", arg);
 #v-
 \seealso{set_prefix_argument}
 \done

http://git.debian.org/?p=pkg-jed/jed.git;a=commitdiff;h=8e7aaf0299e0d1a3063454b568b9c918548d6474
Author: Jörg Sommer <joerg@xxxxxxxxxxxx>
Date:   Sun Aug 24 17:41:40 2008 +0200

    Make return type of jed_malloc0() void*

    The cast from char* to any other pointer type might increases required
    alignment of target type. On some architectures (like alpha) this
    triggers an error message at runtime, if the access of a pointer is
    unaligned:

    % cat cast-test.c
    int main(void)
    {
        char buf[10] = { 0 };
        int *a = (int*)&buf[1];
        *a = 0x7a7a7a;

        return 0;
    }
    % gcc -Wall -Wcast-align cast-test.c -o cast-test
    cast-test.c: In function `main':
    cast-test.c:4: warning: cast increases required alignment of target type
    % ./cast-test
    Unaligned access pid=398552 <cast-test> va=0x11fffbfe1 pc=0x12000112c ra=0x1200010ac inst=0xb0220000

M       src/abbrev.c
M       src/blocal.c
M       src/buffer.c
M       src/hooks.c
M       src/indent.c
M       src/menu.c
M       src/misc.c
M       src/misc.h
M       src/paste.c
M       src/screen.c
M       src/vterm.c
M       src/window.c

http://git.debian.org/?p=pkg-jed/jed.git;a=commitdiff;h=222c5d0c856c5cf1eb218493eddbf33d6e43fd5c
Author: Jörg Sommer <joerg@xxxxxxxxxxxx>
Date:   Sun Aug 24 18:34:57 2008 +0200

    Use intermediate void* pointer for SLmalloc() return values

    The cast from char* to any other pointer type might increases required
    alignment of target type. This causes problems on some architectures
    (like alpha).

    Fix it by saving the result of SLmalloc() and SLrealloc() to a void*
    pointer and then copy it to the target variable.

M       src/abbrev.c
M       src/buffer.c
M       src/dfasyntx.c
M       src/hooks.c
M       src/ledit.c
M       src/menu.c
M       src/undo.c
M       src/vfile.c
M       src/vterm.c
M       src/xterm.c

Bye, Jörg.
-- 
„Dass man etwas durchdringen kann, wenn man es durchschaut
 hat, ist der Irrtum der Fliege an der Fensterscheibe.“ (Nietzsche)

--------------------------
To unsubscribe send email to <jed-users-request@xxxxxxxxxxx> with
the word "unsubscribe" in the message body.
Need help? Email <jed-users-owner@xxxxxxxxxxx>.