Refactor error handling in Base64LineBreaker

Switched ErrNoOutWriter from a string constant to an error object for improved error handling consistency. This change enhances clarity and reduces the risk of error handling inconsistencies in the future.
This commit is contained in:
Winni Neessen 2024-10-23 11:16:09 +02:00
parent 2088796049
commit 946d9888d6
Signed by: wneessen
GPG key ID: 385AC9889632126E

View file

@ -13,8 +13,8 @@ import (
// in encoding processes. // in encoding processes.
var newlineBytes = []byte(SingleNewLine) var newlineBytes = []byte(SingleNewLine)
// ErrNoOutWriter is the error message returned when no io.Writer is set for Base64LineBreaker. // ErrNoOutWriter is the error returned when no io.Writer is set for Base64LineBreaker.
const ErrNoOutWriter = "no io.Writer set for Base64LineBreaker" var ErrNoOutWriter = errors.New("no io.Writer set for Base64LineBreaker")
// Base64LineBreaker handles base64 encoding with the insertion of new lines after a certain number // Base64LineBreaker handles base64 encoding with the insertion of new lines after a certain number
// of characters. // of characters.
@ -44,7 +44,7 @@ type Base64LineBreaker struct {
// - err: An error if one occurred during the write operation. // - err: An error if one occurred during the write operation.
func (l *Base64LineBreaker) Write(data []byte) (numBytes int, err error) { func (l *Base64LineBreaker) Write(data []byte) (numBytes int, err error) {
if l.out == nil { if l.out == nil {
err = errors.New(ErrNoOutWriter) err = ErrNoOutWriter
return return
} }
if l.used+len(data) < MaxBodyLength { if l.used+len(data) < MaxBodyLength {