-
Notifications
You must be signed in to change notification settings - Fork 80
Closed
Labels
conclusion: resolvedIssue was resolvedIssue was resolvedtopic: codeRelated to content of the project itselfRelated to content of the project itselftype: imperfectionPerceived defect in any part of projectPerceived defect in any part of project
Description
Hello
While debugging an (unrelated) issue I had a look at the source code of the library and found this in the end
function in RS485.cpp (line 75 and following):
if (_rePin > -1) {
digitalWrite(_rePin, LOW);
pinMode(_dePin, INPUT);
}
if (_dePin > -1) {
digitalWrite(_dePin, LOW);
pinMode(_rePin, INPUT);
}
I think the _dePin
and _rePin
in the pinMode functions are swapped.
I assume this is not a real issue, since not a lot of people would use the end function, but for completeness sake maybe it is to fix
Metadata
Metadata
Assignees
Labels
conclusion: resolvedIssue was resolvedIssue was resolvedtopic: codeRelated to content of the project itselfRelated to content of the project itselftype: imperfectionPerceived defect in any part of projectPerceived defect in any part of project