Swss common exceptions

[erer1243]

First 4 commits are from swss-common table pr :P

Closes #10

[mssonicbld]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

Copilot reviewed 5 out of 16 changed files in this pull request and generated no comments.

Files not reviewed (11)

• crates/swss-common/src/types/zmqserver.rs: Evaluated as low risk
• crates/swss-common/src/types/zmqconsumerstatetable.rs: Evaluated as low risk
• crates/swss-common/tests/common.rs: Evaluated as low risk
• crates/swss-common/tests/async.rs: Evaluated as low risk
• crates/swss-common/src/types/table.rs: Evaluated as low risk
• crates/swss-common/src/types/zmqproducerstatetable.rs: Evaluated as low risk
• crates/swss-common/src/types/subscriberstatetable.rs: Evaluated as low risk
• crates/swss-common/src/types/zmqclient.rs: Evaluated as low risk
• crates/swss-common/src/types/cxxstring.rs: Evaluated as low risk
• crates/swss-common/src/lib.rs: Evaluated as low risk
• crates/swss-common/src/types/consumerstatetable.rs: Evaluated as low risk

Comments suppressed due to low confidence (6)

crates/swss-common/src/types/producerstatetable.rs:12

• [nitpick] The function name new is ambiguous. It should be renamed to new_with_result.

pub fn new(db: DbConnector, table_name: &str) -> Result<Self> {

crates/swss-common/src/types/producerstatetable.rs:18

• [nitpick] The function name set_buffered is ambiguous. It should be renamed to set_buffered_with_result.

pub fn set_buffered(&self, buffered: bool) -> Result<()> {

crates/swss-common/src/types/dbconnector.rs:30

• The error message in the DbConnector::new function should be consistent with the rest of the codebase.

Exception::try1(|p_db| SWSSDBConnector_new_tcp(db_id, hostname.as_ptr(), *port, timeout_ms, p_db))?

crates/swss-common/src/types/dbconnector.rs:36

• The error message in the DbConnector::new function should be consistent with the rest of the codebase.

Exception::try1(|p_db| SWSSDBConnector_new_unix(db_id, sock_path.as_ptr(), timeout_ms, p_db))?

crates/swss-common/src/types/dbconnector.rs:83

• The error message in the DbConnector::set function should be consistent with the rest of the codebase.

unsafe { Exception::try0(SWSSDBConnector_set(self.ptr, key.as_ptr(), val.as_raw())) }

crates/swss-common/src/types/dbconnector.rs:113

• The error message in the DbConnector::hset function should be consistent with the rest of the codebase.

Exception::try0(SWSSDBConnector_hset(self.ptr, key.as_ptr(), field.as_ptr(), val.as_raw()))

[erer1243]

Those copilot reviews don't really make sense