With new jucer, windows.h is not only unnecesary, but can cause conflicts with some definitions.
Also, OpenGL.h includes disabled, as it's no longer needed and it, by itself, needs windows.h

The DataViewport was changing the size of Visualizers,
even when they were placed in their own DocumentWindow,
because "destroyTab" was only removing the tab from the
TabbedButtonBar, rather than the TabbedComponent.

The Visualizer class no longer inherents from OpenGLCanvas. In the future,
scrolling will be handled by a Viewport object.

Using ScopedPointers was causing memory leaks. It's worth tracking
down the source of these in the future, but for now, we'll leave
the code as it was before.

Use the following command to update the copyright year in all files,
starting in the Source/ directory:

find . -type f -print0 | xargs -0
   sed -i 's/Copyright (C) 2012/Copyright (C) 2013/g'

Works like a charm!

When many channels are selected for recording a race condition can cause a crash by writing to a file that has not been opened yet.

It's unused, declared obsolete, and is triggering GCC warnings. If
anyone wants it back, it'll be in the Git history.

Signed-off-by: Marti Bolivar <mbolivar@leaflabs.com>

Signed-off-by: Marti Bolivar <mbolivar@leaflabs.com>

Signed-off-by: Marti Bolivar <mbolivar@leaflabs.com>

getInputChannelName() and getOutputChannelName() (pure virtual in the
superclass) aren't returning anything. So far this hasn't triggered
any memory errors because nobody's calling them.

To hack around this, have both of these return
"xxx-UNUSED-OPEN-EPHYS-xxx" (none of the existing subclases override
these methods, so leaving them pure virtual isn't workable). This way,
if we ever start calling these, we'll notice right away that a fix is
needed (instead of segfaulting, etc.).

Also keep them virtual to allow overrides.

Signed-off-by: Marti Bolivar <mbolivar@leaflabs.com>

That's not gonna work. Since nullParam is just a "nothing found", put
it in static storage and return that.

Signed-off-by: Marti Bolivar <mbolivar@leaflabs.com>

It looks like declaring them to return bool was a mistake:

1. None of their callers check their return value.

2. They don't return a value, and it doesn't look like there's any
   reason for them to.

Signed-off-by: Marti Bolivar <mbolivar@leaflabs.com>

If it's for internal use only, then enforce that.

Signed-off-by: Marti Bolivar <mbolivar@leaflabs.com>

Fixes caught by -Wreturn-type.

Have enable() properly return a value. Have disable() actually disable
the Arduino, and then properly return a value.

Signed-off-by: Marti Bolivar <mbolivar@leaflabs.com>

Signed-off-by: Marti Bolivar <mbolivar@leaflabs.com>

Generic changes:

- Genuinely unused variables (-Wunused-variable): delete
  them. Sometimes add a comment if the variable is for an extension of
  the code that isn't done yet.

- Variables that became unused due to commented-out
  code (-Wunused-variable): comment out their definitions. Dirty, but
  that's because commenting out code instead of deleting it is
  dirty.

  Many of these are due to commented-out printlines that should have
  been log messages that only get emitted at high verbosity level.

Specific changes:

- SpikeObject.cpp (-Wunused-value): move initialization of loop
  variable into first for loop clause to silence the warning.

Signed-off-by: Marti Bolivar <mbolivar@leaflabs.com>

Signed-off-by: Marti Bolivar <mbolivar@leaflabs.com>