easylinux/NmraDcc
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fixed issue reported by lucari

Browse Source 
Despite the values set for F0-4, a packet with all fields set to zero is received. The problem seems to be related to NrmaDcc.cpp and a hidden 'break' statement at line 484. This should be swapped with the #endif on the next line. The result is that when a Speed packet is processed, it drops through to the next case statement and sends an empty DCCFunc command.
This commit is contained in:
Alex Shepherd
2016-05-11 20:53:06 +12:00
parent f50fea1a85
commit ebfb58c1fb
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
NmraDcc.cpp
View File

@@ -481,8 +481,8 @@ void processMultiFunctionMessage( uint16_t Addr, DCC_ADDR_TYPE AddrType, uint8_t
uint8_t fn0 = (Cmd & 0b00010000) ; uint8_t fn0 = (Cmd & 0b00010000) ;
notifyDccFunc( Addr, AddrType, FN_0, fn0 ) ; notifyDccFunc( Addr, AddrType, FN_0, fn0 ) ;
} }
break;
#endif #endif
break;
case 0b10000000: // Function Group 0..4 case 0b10000000: // Function Group 0..4
if( notifyDccFunc ) if( notifyDccFunc )