Re: Other formats in pset like markdown, rst, mediawiki - Mailing list pgsql-hackers
From | Jan Michálek |
---|---|
Subject | Re: Other formats in pset like markdown, rst, mediawiki |
Date | |
Msg-id | CAAYBy8Y_C1O9GvbWRTXvug9J6mKC=dTRDN3Wun9oaszp0qjfgA@mail.gmail.com Whole thread Raw |
In response to | [HACKERS] Other formats in pset like markdown, rst, mediawiki (Jan Michálek <godzilalalala@gmail.com>) |
Responses |
Re: Other formats in pset like markdown, rst, mediawiki
(Pierre Ducroquet <p.sql@pinaraf.info>)
|
List | pgsql-hackers |
2017-03-30 21:53 GMT+02:00 Pavel Stehule <pavel.stehule@gmail.com>:
psql (10devel, server 9.6.1)
Type "help" for help.
jelen=# \x
Expanded display is on.
jelen=# \pset format rst
Output format is rst.
jelen=# select 10;
+--------------+----+
| **RECORD 1** |
+--------------+----+
| **?column?** | 10 |
+--------------+----+
jelen=#
2017-03-29 20:11 GMT+02:00 Jan Michálek <godzilalalala@gmail.com>:2017-03-27 19:41 GMT+02:00 Jan Michálek <godzilalalala@gmail.com>:2017-03-23 17:26 GMT+01:00 Pierre Ducroquet <p.psql@pinaraf.info>:The following review has been posted through the commitfest application:
make installcheck-world: tested, passed
Implements feature: tested, passed
Spec compliant: tested, passed
Documentation: tested, passed
Hi
This is my first review (Magnus said in his presentation in PGDay Paris that volunteers should just come and help, so here I am), so please notify me for any mistake I do when using the review tools...
The feature seems to work as expected, but I don't claim to be a markdown and rst expert.
Some minor issues with the code itself :
- some indentation issues (documentation and code itself with mix between space based and tab based indentation) and a few trailing spaces in codecorrected- typographic issues in the documentation :
- "The html, asciidoc, latex, latex-longtable, troff-ms, and markdown and rst formats" ==> duplicated andcorrected- "Sets the output format to one of unaligned, aligned, wrapped, html, asciidoc, latex (uses tabular), latex-longtable, rst, markdown, or troff-ms." ==> extra comma at the end of the list
- the comment " dont add line after last row, because line is added after every row" is misleading, it should warn that it's only for rst
- there is a block of commented out code left
- in the print_aligned_vertical function, there is a mix between "cont->opt->format == PRINT_RST" and "format == &pg_rst" and I don't see any obvious reason for thatcorrected- the documentation doesn't mention (but ok, it's kind of obvious) that the linestyle option will not work with rst and markdownIn this patch are corrected (i hope, i had correct changes in vimrc) indentation issues. Plese, look at this if it is OK (i men indentats) and some minor errors. And it should work on current master (probably).
Added \x option form markdownIn markdown works multiline cels (newline replaced by </br>)regre tests passed\pset format rst\xselect 10crash on segfaultProgram received signal SIGSEGV, Segmentation fault.0x00007f77673a866c in vfprintf () from /lib64/libc.so.6(gdb) bt#0 0x00007f77673a866c in vfprintf () from /lib64/libc.so.6#1 0x00007f77673b1574 in fprintf () from /lib64/libc.so.6#2 0x0000000000437bc5 in print_aligned_vertical (cont=0x7fffade43da0, fout=<optimized out>,is_pager=<optimized out>) at print.c:1755#3 0x000000000043a70d in printTable (cont=cont@entry=0x7fffade43da0, fout=<optimized out>, fout@entry=0x7f77677255e0 <_IO_2_1_stdout_>, is_pager=<optimized out>, is_pager@entry=0 '\000',flog=flog@entry=0x0) at print.c:3466#4 0x000000000043c37f in printQuery (result=result@entry=0x9c4b60, opt=opt@entry=0x7fffade43f00,fout=0x7f77677255e0 <_IO_2_1_stdout_>, is_pager=is_pager@entry=0 '\000', flog=0x0) at print.c:3551#5 0x000000000040da6d in PrintQueryTuples (results=0x9c4b60) at common.c:808#6 PrintQueryResults (results=0x9c4b60) at common.c:1140#7 SendQuery (query=0x9c1700 "select 10;") at common.c:1317#8 0x000000000041c3d4 in MainLoop (source=0x7f77677248a0 <_IO_2_1_stdin_>) at mainloop.c:319#9 0x0000000000405d5d in main (argc=<optimized out>, argv=<optimized out>) at startup.c:396Regards
On source from monday it works (last commit on master I have is from 27.3 14:30). Or, maybe I didn`t generate diff well, or some gitt issue.
psql (10devel, server 9.6.1)
Type "help" for help.
jelen=# \x
Expanded display is on.
jelen=# \pset format rst
Output format is rst.
jelen=# select 10;
+--------------+----+
| **RECORD 1** |
+--------------+----+
| **?column?** | 10 |
+--------------+----+
jelen=#
PavelJan
Have nice dayJanThanks !
The new status of this patch is: Waiting on Author
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers
--JelenStarší čeledín datovýho chlíva
--JelenStarší čeledín datovýho chlíva
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers
--
Jelen
Starší čeledín datovýho chlívapgsql-hackers by date: