mpogue2 / SquareDesk

Fully-featured music player and sequence designer, designed for square dance callers
10 stars 4 forks source link

Feature: refactor to use a single CSV line parser #148

Open mpogue2 opened 7 years ago

mpogue2 commented 7 years ago

Did you want to refactor to use MainWindow::parseCSV() everywhere (or alternately, to get rid of parseCSV and use your splitCSV() function everywhere from importdialog.cpp? I'm agnostic as to which is better -- I haven't really looked at each of them in detail.

danlyke commented 7 years ago

I haven't looked at either either. I suspect MainWindow::parseCSV is better, but we'll look.