Attention is currently required from: neels, laforge, keith.
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-msc/+/28114 )

Change subject: WIP: switch from libdbi to lbsqlite3
......................................................................


Patch Set 8:

(3 comments)

Commit Message:

https://gerrit.osmocom.org/c/osmo-msc/+/28114/comment/839b3016_be3a276c
PS8, Line 7: WIP: switch from libdbi to lbsqlite3
Is it still WIP?


File include/osmocom/msc/db.h:

https://gerrit.osmocom.org/c/osmo-msc/+/28114/comment/f9610b13_2f83cb08
PS8, Line 43:                                  int max_failed);
does it make sense to pass negative values here?


File src/libmsc/db.c:

https://gerrit.osmocom.org/c/osmo-msc/+/28114/comment/87c1e049_d2dd35eb
PS8, Line 489:  break;
wrong indentation



--
To view, visit https://gerrit.osmocom.org/c/osmo-msc/+/28114
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-msc
Gerrit-Branch: master
Gerrit-Change-Id: Ia334904289f92d014e7bd16b02b3b5817c12c790
Gerrit-Change-Number: 28114
Gerrit-PatchSet: 8
Gerrit-Owner: laforge <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: keith <[email protected]>
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-Reviewer: neels <[email protected]>
Gerrit-Reviewer: pespin <[email protected]>
Gerrit-Attention: neels <[email protected]>
Gerrit-Attention: laforge <[email protected]>
Gerrit-Attention: keith <[email protected]>
Gerrit-Comment-Date: Tue, 17 May 2022 09:22:37 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment

Reply via email to