[HECnet] New bridge.c feedback

Johnny Billquist bqt at softjar.se
Thu Jun 7 09:11:21 PDT 2012


On 2012-06-07 09:42, Johnny Billquist wrote:
Hi, Alastair. Thanks for the feedback. Interesting... Care to help me
figure some things out...?

On 2012-06-07 08:33, Boyanich, Alastair wrote:
cc-1171 cc: WARNING File = bridge.c, Line = 684
The indicated expression has no effect.

if (port == 0) port == DPORT;
^

Blech! Well, DPORT is by default set to 0. See the documentation of the
define in the sources... This one will stay.

Oh well. I made a conditional around this, so that if people don't change DPORT, it will not even be compiled. Warning gone...

Cleaned up the code some, based on inspiration fallout here. New version available. No functional changes...

	Johnny



More information about the Hecnet-list mailing list