Add support for changing the loopback flag

[jjguti]

Small PR to allow the local loopback to be disabled/enabled on a socketcan interface.

[codecov]

Codecov Report

Merging #960 (51edc8a) into develop (7ddea9e) will decrease coverage by 0.51%.
The diff coverage is 60.00%.

@@             Coverage Diff             @@
##           develop     #960      +/-   ##
===========================================
- Coverage    70.78%   70.27%   -0.52%     
===========================================
  Files           75       76       +1     
  Lines         7267     7401     +134     
===========================================
+ Hits          5144     5201      +57     
- Misses        2123     2200      +77     

[felixdivo]

@jjguti This looks like a neat addition. But I think we should add a test to make sure it correctly loops back and only does so if the flag is set. Could You add that? Then, this can be merged.

[jjguti]

@felixdivo Hi, I have now added a test for the local_loopback flag, please tell me if it's good enough for a merge or if something else should be added.

[felixdivo]

Thanks for your quick reply! And sorry for the long pause, there is recently quite a backlog of issues/PRs.

I think that the test is already nearly complete, but we should add a few lines, see my comments below.

[hardbyte]

👍🏼

[felixdivo]

Nice @jjguti!