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

Change subject: db: Switch from 'synchronous = FULL' to 'synchronous = NORMAL'
......................................................................


Patch Set 1: Code-Review+1

(1 comment)

File src/libmsc/db.c:

https://gerrit.osmocom.org/c/osmo-msc/+/28130/comment/f49b8c04_4d9008c2
PS1, Line 566:  if (rc != SQLITE_OK) {
typo: lol it didn't fail?

Also maybe use same format, " = " or "=" for both.



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

Gerrit-Project: osmo-msc
Gerrit-Branch: master
Gerrit-Change-Id: Ie782f0fe90e7204c4d55cdb3948b728c348367d1
Gerrit-Change-Number: 28130
Gerrit-PatchSet: 1
Gerrit-Owner: laforge <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin <[email protected]>
Gerrit-Attention: laforge <[email protected]>
Gerrit-Comment-Date: Tue, 17 May 2022 09:43:27 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment

Reply via email to