This is an automated email from the git hooks/post-receive script.

guillem pushed a commit to branch main
in repository dpkg.

View the commit online:
https://git.dpkg.org/cgit/dpkg/dpkg.git/commit/?id=9fa852980593e27c9e66720df235125940e447bb

commit 9fa852980593e27c9e66720df235125940e447bb
Author: Guillem Jover <[email protected]>
AuthorDate: Tue Jul 23 02:46:00 2024 +0200

    libdpkg: Remove varbuf C++ operator() methods
    
    These are very confusing as they resemble constructors. We have switched
    the code to use operator+= methods instead which make the code and
    intention way more clear.
---
 lib/dpkg/varbuf.h | 14 --------------
 1 file changed, 14 deletions(-)

diff --git a/lib/dpkg/varbuf.h b/lib/dpkg/varbuf.h
index 088d4b9f7..2738547e4 100644
--- a/lib/dpkg/varbuf.h
+++ b/lib/dpkg/varbuf.h
@@ -102,8 +102,6 @@ struct varbuf {
        varbuf &operator+=(const varbuf &v);
        varbuf &operator+=(int c);
        varbuf &operator+=(const char *str);
-       void operator()(int c);
-       void operator()(const char *s);
        const char *str();
 #endif
 };
@@ -386,18 +384,6 @@ varbuf::operator+=(const char *str)
        return *this;
 }
 
-inline void
-varbuf::operator()(int c)
-{
-       varbuf_add_char(this, c);
-}
-
-inline void
-varbuf::operator()(const char *s)
-{
-       varbuf_add_str(this, s);
-}
-
 inline const char *
 varbuf::str()
 {

-- 
Dpkg.Org's dpkg

Reply via email to