search

TelluIoT / ThingML

The ThingML modelling language
https://github.com/TelluIoT/ThingML
Apache License 2.0
102 stars 32 forks source link

Disallow multiple connectors with same pair of required-provided ports #153

Closed jakhog closed 6 years ago

jakhog commented 7 years ago

Currently the following code does not produce any errors

connector someThing.Port => otherThing.Port
connector someThing.Port => otherThing.Port

nor does it for external connectors

connector someThing.Port over Protocol
connector someThing.Port over Protocol

In my mind, this should not be valid ThingML code, and should not be allowed by the compiler.

If it is supposed to be valid, I'm not quite sure what it means. Should you send/receive multiple copies of the same messages?

brice-morin commented 7 years ago

IMHO, n-to-m connectors should be allowed except in your very case.