Report Firmware not working in Windows

[soundanalogous]

When the StandardFirmata (v2.3.1 that comes with arduino v1.0) gets compiled and uploaded to an Uno R1 board then if you try to get the REPORT_FIRMWARE it retuns nothing. The problem is located on line 123 of the Firmata.cpp (filename =
strrchr(name, '/') + 1;) and it is caused because I am compiling the .ino and .cpp under Windows 7 (so the FILE is something like C:\Robot\ ... \Firmata.cpp instead of C:/Robot/ ... /Firmata.cpp as expected).

[soundanalogous]

Something like this should work. _WIN32 for be for any compiler on the Win32 platform (MSVC, MinGW, etc). If the problem is only related to the MSVC compiler, then it would be #ifdef _MSC_VER.

#ifdef _WIN32
strrchr(name, '') + 1;
#else
strrchr(name, '/') + 1;
#endif

.hc

[soundanalogous]

How to handle 64-bit Windows?

[basile-laderchi]

On the work_machine (see P.S.) this code is not working:

#if defined WIN32 || defined WIN64 || defined _WIN32 || defined _WIN64 || defined __CYGWIN__ || defined __MINGW32__
  filename = strrchr(name, '\\') + 1;
#else
  filename = strrchr(name, '/') + 1;
#endif

The only thing that **worked** was replacing this:

filename = strrchr(name, '/') + 1;

with this:

filename = strrchr(name, '\\') + 1;

As for 64-bit Windows I found this: According to [msdn](http://msdn.microsoft.com/en-us/library/b0084kay.aspx) the _WIN32 should be "Defined for applications for Win32 and Win64. Always defined.", but this "Most compilers targeting Microsoft Windows implicitly define _WIN32.[1] This allows code, including preprocessor commands, to compile only when targeting Windows systems. A few compilers define WIN32 instead. For such compilers that do not implicitly define the _WIN32 macro, it can be specified on the compiler's command line, using -D_WIN32." [Wikipedia](http://en.wikipedia.org/wiki/C_preprocessor#Conditional_compilation) has to be taken into consideration too.

basile

P.S.: I have in my possession two machines:
home_machine: Win7 32bit with VS2010 and arduino v1.0 and
work_machine: Win7 64bit with VS2010 and arduino v1.0.

[soundanalogous]

would something like this work?

if (strrchr(name, '/') != NULL) filename = strrchr(name, '/') + 1;
else filename = strrchr(name, '\\') + 1;

[basile-laderchi]

It works.
My Firmata.cpp has the following change and is reporting back "StandardFirmata" v2.3 (with StandardFirmata.ino loaded on the Arduino):

  if (strrchr(name, '/') != NULL)
    filename = strrchr(name, '/') + 1; //points to slash, +1 gets to start of filename
  else
    filename = strrchr(name, '\\') + 1; //points to backslash, +1 gets to start of filename

basile

[adamlast]

I found I had to do this as well on my Win7 64 bit machine. This is an issue that's be known about for 5 months but not fixed.

Is it actually really sensible to generate the Firmware string dynamically during run time based on file name? What are the advantages?

[kisoft]

I edited a method "setFirmwareNameAndVersion":

void FirmataClass::setFirmwareNameAndVersion(const char *name, byte major, byte minor)
{
  const char *filename;
  const char *extension;
  int len;

  extension = strrchr(name,'.');
  filename = strrchr(name,'/');
  if(!filename)
  {
    filename = strrchr(name,'\\');
  }
  if(!filename)
  {
    filename = name;
  }
  else
  {
    filename++;
  }
  if(!extension)
  {
    len =strlen( filename );
  }
  else
  {
    len = extension - filename;
  }
  firmwareVersionCount = len + 2;
  firmwareVersionVector = (byte *) malloc(firmwareVersionCount+1);
  firmwareVersionVector[0] = major;
  firmwareVersionVector[1] = minor;
  strncpy((char*)firmwareVersionVector + 2, filename, len);
  firmwareVersionVector[firmwareVersionCount] = '\0';
}

It's worked on my MSVC 2010 and Arduino Leonardo.
Good Luck!

[soundanalogous]

updated in dev branch