splatch commented on pull request #201:
URL: https://github.com/apache/plc4x/pull/201#issuecomment-818295134


   The SPI is not an valid point, cause within SPI we have things such as 
`PlcReader`, `PassiveDiscovery` and other implementation details which are 
relevant from *driver* point of view. One of very few things related to use of 
API is `PlcProprietaryRequest` which is different from what I am asking about.
   I do not consider calling specific field types in context of specific PLC 
driver a pollution. I believe string concatenation can do much more harm and 
requires much more testing.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to