[elmo-users] core i uwagi

[Szymon]

Cześć. Znuf core. (uwagi na końcu)

to jest elmo ,,stabilne'' 083

$ gdb elmo core.7456 
GNU gdb 5.3
Copyright 2002 Free Software Foundation, Inc.
GDB is free software, covered by the GNU General Public License, and you are
welcome to change it and/or distribute copies of it under certain conditions.
Type "show copying" to see the conditions.
There is absolutely no warranty for GDB.  Type "show warranty" for details.
This GDB was configured as "i386-slackware-linux"...
Core was generated by `elmo'.
Program terminated with signal 11, Segmentation fault.
Reading symbols from /lib/libpthread.so.0...done.
Loaded symbols for /lib/libpthread.so.0
Reading symbols from /lib/libncurses.so.5...done.
Loaded symbols for /lib/libncurses.so.5
Reading symbols from /lib/libc.so.6...done.
Loaded symbols for /lib/libc.so.6
Reading symbols from /lib/ld-linux.so.2...done.
Loaded symbols for /lib/ld-linux.so.2
#0  0x4012e2df in strlen () from /lib/libc.so.6
(gdb) bt
#0  0x4012e2df in strlen () from /lib/libc.so.6
#1  0xbffff9d4 in ?? ()
#2  0x0806cf57 in put_string_fs (s=0xbffff7c0, t=0x6f00 <Address 0x6f00 out of 
bounds>, fs=0xbffff770) at eprintf.c:226
#3  0x0806d0ce in put_mail_special (s=0xbffff7c0, fmt=0x807cc8a "%016f (%-06S)  
", mail=0x80f58a8) at eprintf.c:290
#4  0x0806d760 in substitute_mail (fmt=0x807cc80 "%?%$%# %D %016f (%-06S)  ", 
mail=0x80f58a8, buf=0x80e0a30 "     Oct 03 ")
    at eprintf.c:519
#5  0x0806d8fc in eprintf_mail (fmt=0x807cc80 "%?%$%# %D %016f (%-06S)  ", 
mail=0x80f58a8, buf=0x80e0a30 "     Oct 03 ") at eprintf.c:603
#6  0x08051f05 in print_line (index=386, win=0x80e8924, y=18, x=0, color=1024) 
at folder.c:150
#7  0x08052574 in fill_page (num=16) at folder.c:309
#8  0x08052627 in put_mails_on_pages () at folder.c:331
#9  0x08052b86 in folder_sort_threads () at folder.c:559
#10 0x08052891 in folder_read_box (name=0x80b7570 "/home/ssbm/mail/testy") at 
folder.c:430
#11 0x0805f1bb in box_selection_hit () at box_selection.c:280
#12 0x0806dfd7 in execute (exec=0x807d580) at exec.c:99
#13 0x0806e186 in exec_run (exec=0x807d580) at exec.c:175
#14 0x0806e37a in execute (k=0x8098498) at keymap.c:73
#15 0x0806e4db in keymap_action (km=0x808ee64, key=13) at keymap.c:138
#16 0x08054a0b in cmd_read_loop () at cmd.c:456
#17 0x08051b1a in main (argc=1, argv=0xbffff9d4) at elmo.c:370
#18 0x400c9bb4 in __libc_start_main () from /lib/libc.so.6

uwagi:

-- prośba: proszę o możliwość korzystania a zewnętrzego czytacza maili. 
   (ponieważ czytacz elmowy jest do dupy(*)), ta prośba *nie* jest związana
   z powyższym core.
   (*) nie chodzi mi o funkcjonalność, tylko o to jak to jest napisane.
   Ja np. nie jestem w stanie czytać Twojego kodu, coby go poprawić bo:
   -- jest zbyt ,,pretensjonalnie'' napisany, coż to jest mój punkt
      widzenia i pewnie się mylę.
   -- robisz rzeczy które wprowadzają zamieszanie i utrudniają
      zrozumienie co się dzieje np:
      -- deklarujesz coś jako '*' a potem dajesz to na wejście funkcji
         która oczekuje '**'
      -- rezerw. pamięć w jakiejś funkcji, wywołujesz z niej inną
         funkcję gdzie coś zwalniasz pod jakimiś mało czytelnymi
         warunkami, a potem w zwalniasz wszystko(w funkcji wywołującej).
         I taki lamer jak ja nie wie czy tam jest po prostu martwy kod,
         czy nie.  Bo o ile ja się orientuję to zwalnianie dwa razy tego
         samego to niedobrze ? (pamiętam że chodziło o regexpy,
         tak przy okazji dlaczego je tak cząsto stosujesz ?)

pozdrawiam, sb