Re: [PATCH 0 of 6 V1] Counters

[Audrius Butkevicius <audrius.butkevicius@xxxxxxxxxxxxxxxx>]

Hi Aris,

On 10/02/2014 16:00, Aris Adamantiadis wrote:
> - Doxygen comments:
> You comments are great. Could you put them in the implementation of
> functions (counters.c) as we do for the other includes in libssh.h ?
> That would be a little more coherent.
I assume you mean put comments in counters.c as well as in libssh.h?
> On the code itself:
> - I think you can merge ssh_bytes_counter_struct and
> ssh_packet_counter_struct. It's very likely that both will be used at
> same time and that reduces the overall complexity. ssh_counter_struct is
> also shorter :) Also means less setters.
> What do you think ?
I can do this, but the issue with this is that in some cases there will 
be no bytes, or no packets.
Furthermore, we can only reduce (for the session) by one argument, 
because you still need to have separate counters for bytes/packets on 
the socket, as well as bytes (but not packets) prior the protocol overhead.
Meaning that I'd still have to store two ssh_counter structs in the 
session, and for the channel I'd be holding a struct which only uses 
bytes (but not packets).


Thanks,
Audrius.