Attention is currently required from: fixeria, pespin.

falconia has posted comments on this change by falconia. ( 
https://gerrit.osmocom.org/c/libosmo-netif/+/42141?usp=email )

Change subject: twjit: add underrun extension feature
......................................................................


Patch Set 2:

(3 comments)

File src/twjit.c:

https://gerrit.osmocom.org/c/libosmo-netif/+/42141/comment/234ed583_004ff8ff?usp=email
 :
PS1, Line 554:          twjit->stats.underruns++;
> In the new iteration of my patch, I added a comment explaining while this 
> stats counter should not b […]
PS2 got CR+1 from @[email protected] with no further comments. Is it OK to 
mark this comment as resolved?


https://gerrit.osmocom.org/c/libosmo-netif/+/42141/comment/53823e88_a25dbff5?usp=email
 :
PS1, Line 630: static void handle_flowing_underrun(struct osmo_twjit *twjit)
> I added an explanatory comment in the new iteration of my patch - is it good 
> enough?
PS2 got CR+1 from @[email protected] with no further comments. Is it OK to 
mark this comment as resolved?


https://gerrit.osmocom.org/c/libosmo-netif/+/42141/comment/401d686a_482c9976?usp=email
 :
PS1, Line 671:          if (read_sb_is_empty(twjit)) {
> if (!read_sb_is_empty(twjit)) { […]
PS2 got CR+1 from @[email protected] with no further comments. Is it OK to 
mark this comment as resolved?



--
To view, visit https://gerrit.osmocom.org/c/libosmo-netif/+/42141?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings?usp=email

Gerrit-MessageType: comment
Gerrit-Project: libosmo-netif
Gerrit-Branch: master
Gerrit-Change-Id: Ibda74c0dbfb163f5d0e3fb13f593a6e2c6817673
Gerrit-Change-Number: 42141
Gerrit-PatchSet: 2
Gerrit-Owner: falconia <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <[email protected]>
Gerrit-Reviewer: pespin <[email protected]>
Gerrit-Attention: pespin <[email protected]>
Gerrit-Attention: fixeria <[email protected]>
Gerrit-Comment-Date: Wed, 18 Feb 2026 17:58:23 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: falconia <[email protected]>
Comment-In-Reply-To: pespin <[email protected]>

Reply via email to