diff --git a/src/kits/app/RosterPrivate.cpp b/src/kits/app/RosterPrivate.cpp index c213d57e71..3e8d9bff5c 100644 --- a/src/kits/app/RosterPrivate.cpp +++ b/src/kits/app/RosterPrivate.cpp @@ -1,24 +1,26 @@ /* - * Copyright 2001-2006, Haiku. + * Copyright 2001-2009, Haiku. * Distributed under the terms of the MIT License. * * Authors: * Ingo Weinhold (bonefish@users.sf.net) */ + //! Access class for BRoster members #include #include + /*! \class BRoster::Private \brief Class used to access private BRoster members. This way, the only friend BRoster needs is this class. */ -// SetTo + /*! \brief Initializes the roster. \param mainMessenger A BMessenger targeting the registrar application. @@ -27,13 +29,13 @@ void BRoster::Private::SetTo(BMessenger mainMessenger, BMessenger mimeMessenger) { - if (fRoster) { + if (fRoster != NULL) { fRoster->fMessenger = mainMessenger; fRoster->fMimeMessenger = mimeMessenger; } } -// SendTo + /*! \brief Sends a message to the registrar. \a mime specifies whether to send the message to the roster or to the @@ -54,19 +56,18 @@ BRoster::Private::SetTo(BMessenger mainMessenger, BMessenger mimeMessenger) status_t BRoster::Private::SendTo(BMessage *message, BMessage *reply, bool mime) { - status_t error = (message ? B_OK : B_BAD_VALUE); - if (error == B_OK && !fRoster) - error = B_NO_INIT; - if (error == B_OK) { - if (mime) - error = fRoster->fMimeMessenger.SendMessage(message, reply); - else - error = fRoster->fMessenger.SendMessage(message, reply); - } - return error; + if (message == NULL) + return B_BAD_VALUE; + if (fRoster == NULL) + return B_NO_INIT; + + if (mime) + return fRoster->fMimeMessenger.SendMessage(message, reply); + + return fRoster->fMessenger.SendMessage(message, reply); } -// IsMessengerValid + /*! \brief Returns whether the roster's messengers are valid. \a mime specifies whether to check the roster messenger or the one of @@ -79,11 +80,11 @@ BRoster::Private::SendTo(BMessage *message, BMessage *reply, bool mime) bool BRoster::Private::IsMessengerValid(bool mime) const { - return (fRoster && (mime ? fRoster->fMimeMessenger.IsValid() - : fRoster->fMessenger.IsValid())); + return fRoster != NULL && (mime ? fRoster->fMimeMessenger.IsValid() + : fRoster->fMessenger.IsValid()); } -// InitBeRoster + /*! \brief Initializes the global be_roster variable. Called before the global constructors are invoked. @@ -94,7 +95,7 @@ BRoster::Private::InitBeRoster() be_roster = new BRoster; } -// DeleteBeRoster + /*! \brief Deletes the global be_roster. Called after the global destructors are invoked.