Small error in commit 740483bb

classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|

Small error in commit 740483bb

Bengt Martensson-2
NEWS now says:

  * plugins: girs: Add udp/receive support (#201).


Even though #201 was entitled "Girs driver should support TCP and UDP",
'only' TCP support was added, but for both sending and receiving.
Suggestion: change to "Add TCP support as alternative to serial connection."

PS. Would UDP make sense here? I am a little reluctant of adding
features "just for completeness". Agile says: "Maximize the amount of
work not done" :-).

Greetz,

Bengt

------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, SlashDot.org! http://sdm.link/slashdot
Reply | Threaded
Open this post in threaded view
|

Re: Small error in commit 740483bb

Alec Leamas


On 12/02/17 12:06, Bengt Martensson wrote:
> NEWS now says:
>
>   * plugins: girs: Add udp/receive support (#201).
>
>
> Even though #201 was entitled "Girs driver should support TCP and UDP",
> 'only' TCP support was added, but for both sending and receiving.
> Suggestion: change to "Add TCP support as alternative to serial connection."

Bug updated, the commit is, well, committed... Also, it doesn't make
much sense  to update NEWS since it's released.
>
> PS. Would UDP make sense here? I am a little reluctant of adding
> features "just for completeness". Agile says: "Maximize the amount of
> work not done" :-).

Seriously, I have no idea. But if there is no use-case, why bother?

Cheers!

--alec

------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, SlashDot.org! http://sdm.link/slashdot