Titor8115 / mandown

man-page inspired Markdown viewer
GNU General Public License v3.0
243 stars 13 forks source link

segfault #16

Closed okbob closed 4 years ago

okbob commented 4 years ago

I try to watch README.md from plpgsql_check project, and it finish by segfault xREADME.md.gz

#0  0x00007ffff7c40a25 in raise () from /lib64/libc.so.6
#1  0x00007ffff7c29895 in abort () from /lib64/libc.so.6
#2  0x00007ffff7c848c7 in __libc_message () from /lib64/libc.so.6
#3  0x00007ffff7c8494a in __libc_fatal () from /lib64/libc.so.6
#4  0x00007ffff7c6d6ca in printf_positional () from /lib64/libc.so.6
#5  0x00007ffff7c6fb50 in __vfprintf_internal () from /lib64/libc.so.6
#6  0x00007ffff7c83aaa in __vsnprintf_internal () from /lib64/libc.so.6
#7  0x00007ffff7f8ffa4 in _nc_printf_string_sp () from /lib64/libncursesw.so.6
#8  0x00007ffff7f89285 in vw_printw () from /lib64/libncursesw.so.6
#9  0x00007ffff7f89418 in wprintw () from /lib64/libncursesw.so.6
#10 0x000000000040316c in render_content (dest=0x4202a0, ib=)
    at src/view.c:186
#11 0x0000000000403b6a in view (configed=configed@entry=0x4122a0, ob=ob@entry=0x4128e0, 
    blocks=) at src/view.c:464
#12 0x00000000004026c8 in main (argc=, argv=)
    at src/mandown.c:180
Titor8115 commented 4 years ago

On it. There's a new version coming up, and it seems to have no problem rendering README.md from plpgsql_check repository. I'll finish polishing and uploading it this weekend. And check out whats wrong with this version.

(The new version featuring a clipboard for individual url reference links.)

Titor8115 commented 4 years ago

@okbob New version has been uploaded. I tested it on plpgsql_check's README.md and worked like a charm. Apparently there's some POSIX issues that caused the seg fault.

The clipboard polishing is taking a bit of time... But it will be done in coming days

okbob commented 4 years ago

po 15. 6. 2020 v 7:41 odesílatel Titor8115 notifications@github.com napsal:

@okbob https://github.com/okbob New version has been uploaded. I tested it on plpgsql_check's README.md and worked like a charm. Apparently there's some POSIX issues that caused the seg fault.

yes - it is working now nice.

Probably for code and pre tag a long lines should be truncated (not wrapped)

But it is nice tool

The clipboard polishing is taking a bit of time... But it will be done in coming days

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/Titor8115/mandown/issues/16#issuecomment-643914498, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAEFO45HPVJKLVXTH2HSH4TRWWYBXANCNFSM4N42AIXQ .

Titor8115 commented 4 years ago

Probably for code and pre tag a long lines should be truncated (not wrapped)

I'll look into that. Initially I was just thinking about the ease of copying. Currently '< pre >' is defined as "don't format me". But you are definitely right about truncates for '< code >' tag.