<< Back to previous view

[CLJ-1009] make print-table org mode compatible Created: 08/Jun/12  Updated: 18/Aug/12  Resolved: 18/Aug/12

Status: Closed
Project: Clojure
Component/s: None
Affects Version/s: None
Fix Version/s: Release 1.5

Type: Enhancement Priority: Minor
Reporter: Stuart Halloway Assignee: Stuart Halloway
Resolution: Completed Votes: 2
Labels: None

Attachments: Text File clj-1009-print-table-org-mode-patch3.txt     Text File print-table-org-mode.txt    
Patch: Code
Approval: Ok

 Comments   
Comment by Andy Fingerhut [ 08/Jun/12 10:39 PM ]

Patch clj-1009-print-table-org-mode-patch2.txt dated June 8, 2012 is the same as Stuart Halloway's print-table-org-mode.txt of the same date, except a couple of tests have been added. His code changes look correct to me.

Comment by Aaron Bedra [ 14/Aug/12 8:22 PM ]

This is not quite correct. the header separator in org-mode is 

|--+--+--|
not 
+--+--+

Comment by Andy Fingerhut [ 15/Aug/12 10:29 AM ]

clj-1009-print-table-org-mode-patch3.txt dated Aug 15, 2012 is same as the previous -patch2.txt version of Jun 8, 2012 (now deleted as obsolete), except it corrects the header separator as identified by Aaron Bedra.

Comment by Andy Fingerhut [ 15/Aug/12 10:31 AM ]

Presumptuously changing approval from Incomplete back to Vetted, as it was before Aaron Bedra described a problem with the patch, and I updated the patch to address his comment. Feel free to apply the smackdown if necessary.

Comment by Aaron Bedra [ 15/Aug/12 10:44 AM ]

Looks ok now. Three cheers for org-mode awesomeness!!!

Generated at Wed Jun 19 05:59:46 CDT 2013 using JIRA 4.4#649-r158309.