Here is a snippet of code that I have been running on Core for months now. The identical program compiles and flashes on Core and on Photon, but the results are different, as described below: Code snippet: if(bufferReadout != "") // skip empty log entries { int index; // parse the comma delim…

I was thinking that either the Time.timeStr() function is behaving differently on Core vs Photon, with something going on with the null termination, or maybe concatenating onto a null-terminated string behaves differently?

I bet @dougal is correct. However my photon hasn’t arrived yet to test…

I’d go for the first one, since doing bufferReadout += "Z (epoch:"; bufferReadout += dest[3]; bufferReadout += "Z) "; bufferReadout += Time.timeStr(index); which does work and still is concatenating onto a null-terminated string (but not a String to a string/String). But since t…

Yeah, it’s been too long since I dealt with the details of C. I thought that regular strings were null-terminated, but I couldn’t remember for sure. So yeah, likely a bug in how += is handled for the String class. I’ve been using loosely-typed languages as a web developer so many years, I have to c…

I tried the change suggested by BDUB. No joy - it is the same. I tried deleting the space before the Z (i.e. "Z (ep… vs " Z (ep… No joy - it is the same. I tired commenting out the string building lines right after bufferReadout += “Z (epoch:”; No joy - same result. I wanted to try some …

@BobG , I guess you tried the suggestion to use c_str(). This function returns a const char* string instead of a String object. While the += operator of the String class taking a const char* is proven to work as expected, the += operator taking a &String or &&String is not. It’s a while since I ch…

Thanks to everyone for their investigative inputs! We clearly have a regression from 0.3.4 to 0.4.0 - I’ll look into this today so we can roll out a fix for 0.4.1 if needed.

Hi @BobG There are two bugs hiding in String right now that I have made Matt @mdma aware of. Here is some work-around code for you test–I am assuming dest[] is declared as int: if(bufferReadout != "") // skip empty log entries { int index; // parse the comma delimited string into its substrin…

Thanks all for the help. I am happy to hear that mdma will look into the regression issue, as the original code works fine for Core but not for Photon. I understand what ScruffR is saying; however, the String class concatenation function is supposed to work (it is documented as such) and thus it i…

Code executes differently on Photon than on Core

Troubleshooting

erjm June 3, 2015, 1:12am 12

Updated Windows Driver for Core and Photon

Topic		Replies	Views
Photon stops responding randomly Troubleshooting	18	2848	November 28, 2017
Code working on Photon doesn't work for Core Troubleshooting photon , core	17	1387	June 16, 2019
Libraries to Update for the Photon Libraries	77	19710	May 20, 2016
Photon Repetitive Connecting to the Cloud Firmware	9	709	July 14, 2020
timeStr() for Core Firmware	8	513	June 10, 2019

Code executes differently on Photon than on Core

Related topics