From df54f93717b0bc701878f34b753228547a2a8479 Mon Sep 17 00:00:00 2001
From: Sunil Nimmagadda <sunil@nimmagadda.net>
Date: Sat, 8 Nov 2025 14:15:00 +0530
Subject: Simplify return value.

Moved away from cursor in commit 55c770ca23575 and wrote directly to a Vec.
No more IO happening and needn't return a Result.
---
 src/vrrpv2.rs | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/vrrpv2.rs b/src/vrrpv2.rs
index 57f692a..b42cba0 100644
--- a/src/vrrpv2.rs
+++ b/src/vrrpv2.rs
@@ -83,7 +83,7 @@ pub enum AuthType {
 }
 
 impl VRRPv2 {
-    pub fn to_bytes(&self) -> Result<Vec<u8>, std::io::Error> {
+    pub fn to_bytes(&self) -> Vec<u8> {
         let sz = MIN_PACKET_SIZE + (self.ip_addrs.len() * 4);
         let mut bytes: Vec<u8> = Vec::with_capacity(sz);
         bytes.push(VERSION_TYPE);
@@ -101,7 +101,7 @@ impl VRRPv2 {
         let cksum_bytes = cksum.to_be_bytes();
         bytes[6] = cksum_bytes[0];
         bytes[7] = cksum_bytes[1];
-        Ok(bytes)
+        bytes
     }
 }
 
@@ -303,6 +303,6 @@ fn test_to_bytes() {
         0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
     ];
     let vrrpv2 = from_bytes(&in_bytes).expect("parsing failed");
-    let out_bytes = vrrpv2.to_bytes().expect("conversion failed");
+    let out_bytes = vrrpv2.to_bytes();
     assert_eq!(in_bytes.as_ref(), out_bytes);
 }
-- 
cgit v1.2.3