Why does XcpSetDaqListMode overwrites flags to be mode?

[f-zl]

I have a question about the function XcpSetDaqListMode in xcpLite.c in commit 0870237 (most recent commit in master). It has this line DaqListFlags(daq) = mode;, which overwrites the flags to be mode.

From the XCP specification (only have version 1.1 in hand), the mode in SET_DAQ_LIST_MODE is bit mask of ALTERNATING, DIRECTION, TIMESTAMP, PID_OFF. The line DaqListFlags(daq) = mode; will clear the bits like DAQ_FLAG_SELECTED and DAQ_FLAG_RUNNING when receiving a SET_DAQ_LIST_MODE command. Is this a bug? Thanks for reply.

[RainerZ]

Thanks. Yes, this is a bug. Unclean code. Unused bits from XCP version 1.2 reused for internal states. Conflict with new DTO_CTR bit in version 1.3. Possible side effect of clearing internal states, depending when SET_DAQ_LIST_MODE is sent. Should be fixed. From: f-zl ***@***.***> Sent: Tuesday, November 7, 2023 8:57 AM To: vectorgrp/XCPlite ***@***.***> Cc: Subscribed ***@***.***> Subject: [vectorgrp/XCPlite] Why does XcpSetDaqListMode overwrites flags to be mode? (Issue #43) I have a question about the function XcpSetDaqListMode in xcpLite.c in commit 0870237<0870237> (most recent commit in master). It has this line DaqListFlags(daq) = mode;, which overwrites the flags to be mode. From the XCP specification (only have verision 1.1 in hand), the mode in SET_DAQ_LIST_MODE is bit mask of ALTERNATING, DIRECTION, TIMESTAMP, PID_OFF. DaqListFlags(daq) = mode; will clear the other bits like DAQ_FLAG_SELECTED and DAQ_FLAG_RUNNING. Is this a bug? Thanks for reply. — Reply to this email directly, view it on GitHub<#43>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/ABAJOU4XDDITI7MVJ4AQG3DYDHSTDAVCNFSM6AAAAAA7AXO4LWVHI2DSMVQWIX3LMV43ASLTON2WKOZRHE4DANZWGE4DGNI>. You are receiving this because you are subscribed to this thread.Message ID: ***@***.******@***.***>>