Myth/serial
1
0
Fork 0
mirror of https://github.com/wjwwood/serial.git synced 2026-01-22 11:44:53 +08:00
Code Issues Projects Releases Wiki Activity

Changed how the memory is allocated in the read operation and changed how the timeouts function. If the timeout is -1, then it will block until it finishes the reads.

Browse Source
This commit is contained in:
John Harrison 2012-01-13 11:33:55 -06:00
parent 6011f2847c
commit f81268fdf0
1 changed files with 33 additions and 17 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

28
src/impl/unix.cc
View File

@ -215,9 +215,19 @@ Serial::SerialImpl::read (size_t size) {
throw PortNotOpenedException("Serial::read");
}
string message = "";
char buf[1024]; // TODO(ash_gti): Should this be 1024? or...?
char *buf = NULL;
// Using size+1 to leave room for a null character
if (size > 1024) {
buf = (char*)malloc((size + 1) * sizeof(*buf));
}
else {
buf = (char*)alloca((size + 1) * sizeof(*buf));
}
fd_set readfds;
memset(buf, 0, (size + 1) * sizeof(*buf));
ssize_t bytes_read = 0;
while (message.length() < size) {
if (timeout_ == -1) {
FD_ZERO(&readfds);
FD_SET(fd_, &readfds);
struct timeval timeout;
@ -232,25 +242,31 @@ Serial::SerialImpl::read (size_t size) {
perror("select()");
exit(EXIT_FAILURE);
}
}
if (FD_ISSET(fd_, &readfds)) {
memset(buf, 0, 1024);
ssize_t bytes_read = ::read(fd_, buf, size-strlen(buf));
if (timeout_ == 1 || FD_ISSET(fd_, &readfds)) {
ssize_t newest_read = ::read(fd_,
buf + bytes_read,
size - static_cast<size_t>(bytes_read));
// read should always return some data as select reported it was
// ready to read when we get to this point.
if (bytes_read < 1) {
if (newest_read < 1) {
// Disconnected devices, at least on Linux, show the
// behavior that they are always ready to read immediately
// but reading returns nothing.
throw SerialExecption("device reports readiness to read but "
"returned no data (device disconnected?)");
}
message.append(buf, (size_t)bytes_read);
bytes_read += newest_read;
}
else {
break; // Timeout
}
}
if (bytes_read > 0)
message.append(buf, (size_t)bytes_read);
if (size > 1024)
free(buf);
return message;
}