[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v2 41/48] hw/net: remove return after g_assert_not_reached()
From: |
Pierrick Bouvier |
Subject: |
[PATCH v2 41/48] hw/net: remove return after g_assert_not_reached() |
Date: |
Thu, 12 Sep 2024 00:39:14 -0700 |
This patch is part of a series that moves towards a consistent use of
g_assert_not_reached() rather than an ad hoc mix of different
assertion mechanisms.
Signed-off-by: Pierrick Bouvier <pierrick.bouvier@linaro.org>
---
hw/net/e1000e_core.c | 2 --
hw/net/igb_core.c | 2 --
hw/net/vmxnet3.c | 1 -
3 files changed, 5 deletions(-)
diff --git a/hw/net/e1000e_core.c b/hw/net/e1000e_core.c
index 248381f9766..2e4c50ddbaf 100644
--- a/hw/net/e1000e_core.c
+++ b/hw/net/e1000e_core.c
@@ -562,7 +562,6 @@ e1000e_rss_calc_hash(E1000ECore *core,
break;
default:
g_assert_not_reached();
- return 0;
}
return net_rx_pkt_calc_rss_hash(pkt, type, (uint8_t *) &core->mac[RSSRK]);
@@ -841,7 +840,6 @@ e1000e_ring_free_descr_num(E1000ECore *core, const
E1000ERingInfo *r)
}
g_assert_not_reached();
- return 0;
}
static inline bool
diff --git a/hw/net/igb_core.c b/hw/net/igb_core.c
index 6be61407715..5dffa12c64b 100644
--- a/hw/net/igb_core.c
+++ b/hw/net/igb_core.c
@@ -398,7 +398,6 @@ igb_rss_calc_hash(IGBCore *core, struct NetRxPkt *pkt,
E1000E_RSSInfo *info)
break;
default:
g_assert_not_reached();
- return 0;
}
return net_rx_pkt_calc_rss_hash(pkt, type, (uint8_t *) &core->mac[RSSRK]);
@@ -747,7 +746,6 @@ igb_ring_free_descr_num(IGBCore *core, const E1000ERingInfo
*r)
}
g_assert_not_reached();
- return 0;
}
static inline bool
diff --git a/hw/net/vmxnet3.c b/hw/net/vmxnet3.c
index 63a91877730..6b5185c707b 100644
--- a/hw/net/vmxnet3.c
+++ b/hw/net/vmxnet3.c
@@ -456,7 +456,6 @@ vmxnet3_setup_tx_offloads(VMXNET3State *s)
default:
g_assert_not_reached();
- return false;
}
return true;
--
2.39.2
- [PATCH v2 29/48] hw/net: remove break after g_assert_not_reached(), (continued)
- [PATCH v2 29/48] hw/net: remove break after g_assert_not_reached(), Pierrick Bouvier, 2024/09/12
- [PATCH v2 30/48] hw/pci-host: remove break after g_assert_not_reached(), Pierrick Bouvier, 2024/09/12
- [PATCH v2 31/48] hw/scsi: remove break after g_assert_not_reached(), Pierrick Bouvier, 2024/09/12
- [PATCH v2 32/48] hw/tpm: remove break after g_assert_not_reached(), Pierrick Bouvier, 2024/09/12
- [PATCH v2 34/48] target/riscv: remove break after g_assert_not_reached(), Pierrick Bouvier, 2024/09/12
- [PATCH v2 35/48] tests/qtest: remove break after g_assert_not_reached(), Pierrick Bouvier, 2024/09/12
- [PATCH v2 33/48] target/arm: remove break after g_assert_not_reached(), Pierrick Bouvier, 2024/09/12
- [PATCH v2 37/48] fpu: remove break after g_assert_not_reached(), Pierrick Bouvier, 2024/09/12
- [PATCH v2 38/48] tcg/loongarch64: remove break after g_assert_not_reached(), Pierrick Bouvier, 2024/09/12
- [PATCH v2 36/48] ui: remove break after g_assert_not_reached(), Pierrick Bouvier, 2024/09/12
- [PATCH v2 41/48] hw/net: remove return after g_assert_not_reached(),
Pierrick Bouvier <=
- [PATCH v2 40/48] hw/hyperv: remove return after g_assert_not_reached(), Pierrick Bouvier, 2024/09/12
- [PATCH v2 39/48] include/qemu: remove return after g_assert_not_reached(), Pierrick Bouvier, 2024/09/12
- [PATCH v2 42/48] hw/pci: remove return after g_assert_not_reached(), Pierrick Bouvier, 2024/09/12
- [PATCH v2 43/48] hw/ppc: remove return after g_assert_not_reached(), Pierrick Bouvier, 2024/09/12
- RE: [PATCH v2 00/48] Use g_assert_not_reached instead of (g_)assert(0, false), Xingtao Yao (Fujitsu), 2024/09/12