This is an automated email from the ASF dual-hosted git repository.

sruehl pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-plc4x.git


The following commit(s) were added to refs/heads/master by this push:
     new a4c4797  [ADS] added TODO: about broken multiplicator
a4c4797 is described below

commit a4c4797f92bd9b079137258d46c3a3941384ff29
Author: Sebastian Rühl <sru...@apache.org>
AuthorDate: Thu Aug 23 10:16:43 2018 +0200

    [ADS] added TODO: about broken multiplicator
---
 .../main/java/org/apache/plc4x/java/ads/protocol/Plc4x2AdsProtocol.java  | 1 +
 1 file changed, 1 insertion(+)

diff --git 
a/plc4j/protocols/ads/src/main/java/org/apache/plc4x/java/ads/protocol/Plc4x2AdsProtocol.java
 
b/plc4j/protocols/ads/src/main/java/org/apache/plc4x/java/ads/protocol/Plc4x2AdsProtocol.java
index c0c0fbf..6e86460 100644
--- 
a/plc4j/protocols/ads/src/main/java/org/apache/plc4x/java/ads/protocol/Plc4x2AdsProtocol.java
+++ 
b/plc4j/protocols/ads/src/main/java/org/apache/plc4x/java/ads/protocol/Plc4x2AdsProtocol.java
@@ -183,6 +183,7 @@ public class Plc4x2AdsProtocol extends 
MessageToMessageCodec<AmsPacket, PlcReque
         Invoke invokeId = Invoke.of(correlationBuilder.incrementAndGet());
         IndexGroup indexGroup = IndexGroup.of(adsAddress.getIndexGroup());
         IndexOffset indexOffset = IndexOffset.of(adsAddress.getIndexOffset());
+        // TODO: multiplicator is missing is missing. So an integer 4Bytes and 
10 of them should be 40 Bytes not one.
         Length length = Length.of(readRequestItem.getSize());
         AmsPacket amsPacket = AdsReadRequest.of(targetAmsNetId, targetAmsPort, 
sourceAmsNetId, sourceAmsPort, invokeId, indexGroup, indexOffset, length);
         LOGGER.debug("encoded read request {}", amsPacket);

Reply via email to