I'm not so sure if this is really a good idea, since now:

write("foo\0bar");

and

char *ptr = "foo\0bar";write(ptr);

Will have different results and I'm not sure that's really a good idea.

Also, Ithink that:

char buf[10];buf[0] = 'X';buf[1] = '\0';write(buf);

will now print 10 bytes, even though only 2 is intended. Since this sort of thing is likely to occur in sketches, this breaks compatibility.

Regarding handling of strings with embedded zeroes, you might be interested in#1936 which improves the String class to better handle them.

Thoughts?

Isn't this what write(buf, length) is supposed to do?

Ideally, both write() functions ought to take (const void *) type, but whether even that can be considered changeable at this late stage in Arduino's development is a good question?

I think that [...] will now print 10 bytes, even though only 2 is intended. Since this sort of thing is likely to occur in sketches, this breaks compatibility.

Yep, you're right - this is not an acceptable change