On Friday 16 May 2008 01:12:14 Uwe Stöhr wrote:
>  > Instead of using our method to decide what is the column delimiter I
>  > have used the csv sniffer (that is the method name, not my fault). :-)
>
> The sniffer works with all my test documents, also the ones generated by
> Excel, but you forgot to enable it by default. I did this in the attached
> version. Sniffing should not be used when the user explicitely set a
> separator via the option. Could you do this please? Then it is ready from
> my point of view.

OK, if no option is given the sniffer is called.

That means that there are no default values, both for the dialect and for the 
column separator.

I have changed the documentation help accordingly.

> Many thanks for looking at the script, it now looks much clearer.

I have cleaned the code a bit more. Now all the strings are defined in the 
begin and the table generation code is IMHO cleaner and easier to read.

> best regards
> Uwe

If this new version works I support its inclusion.
-- 
José Abílio

Attachment: csv2lyx.py
Description: application/python

Reply via email to