FreeEMS Issues - OLV
View Issue Details
0000515OLVGeneral Featurespublic2012-01-27 00:072012-02-03 21:00
0000515: CSV parser fails
See trace, CSV code borks on the file linked here: [^]

You have to manually remove the first line, however apparently all/most ms logs have it, so we should probably make some provision for handling that too. Right now, though, the bug, later the feature.

Smaller file of the three loads OK, other two do not, hogging CPU for ages, then giving the exception below.
Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
    at org.diyefi.openlogviewer.genericlog.GenericDataElement.findMinAndMaxValues(
    at org.diyefi.openlogviewer.genericlog.GenericDataElement.getMinValue(
    at org.diyefi.openlogviewer.optionpanel.OptionFrameV2$ActiveHeaderLabel.setToolTipTextFinal(
    at org.diyefi.openlogviewer.optionpanel.OptionFrameV2$ActiveHeaderLabel.getToolTipText(
    at javax.swing.ToolTipManager$insideTimerAction.actionPerformed(
    at javax.swing.Timer.fireActionPerformed(
    at javax.swing.Timer$
    at java.awt.event.InvocationEvent.dispatch(
    at java.awt.EventQueue.dispatchEventImpl(
    at java.awt.EventQueue.access$000(
    at java.awt.EventQueue$
    at java.awt.EventQueue$
    at Method)
    at java.awt.EventQueue.dispatchEvent(
    at java.awt.EventDispatchThread.pumpOneEventForFilters(
    at java.awt.EventDispatchThread.pumpEventsForFilter(
    at java.awt.EventDispatchThread.pumpEventsForHierarchy(
    at java.awt.EventDispatchThread.pumpEvents(
    at java.awt.EventDispatchThread.pumpEvents(
No tags attached.

2012-01-27 00:09   
It still has one core on my box, I need to kill it, so it's found itself an infinite loop too :-)
2012-01-31 20:50   
(edited on: 2012-01-31 20:51)
What format are these? These don't look like CSV files to me. =/
Are they tab delimited maybe?

2012-01-31 21:54   
The 'C' in CSV is a very loosly used term :-) C can stand for comma, or tab, or numerous other things. I think MTX supports Tab and Comma only. The null thing, as discussed is a MegaJizzism, which we should support, by opening a new issue and eventually adding some hacky code to filter it out.
2012-01-31 22:22   
Such a silly problem. One line code fix though. All done.

Git hash: 72863779a91f201ef805f15ac2f02e04ae66c5b7

This highlighted another issue though which I'll add ASAP.
2012-02-03 21:00   
Fixed, and seems to work with "null" in the first line too!