mirror of
https://github.com/RetroShare/RetroShare.git
synced 2025-02-17 21:34:10 -05:00
Fix Clang warnings: Add author and date to #warning
This commit is contained in:
parent
039908b2c6
commit
e83104ee0f
@ -1972,7 +1972,7 @@ void RsGxsNetService::updateServerSyncTS()
|
|||||||
#endif
|
#endif
|
||||||
|
|
||||||
// I keep the creation, but the data is not used yet.
|
// I keep the creation, but the data is not used yet.
|
||||||
#warning disabled this, but do we need it?
|
#warning csoler 2016-12-12: Disabled this, but do we need it?
|
||||||
// RsGxsServerMsgUpdate& msui(mServerMsgUpdateMap[grpId]) ;
|
// RsGxsServerMsgUpdate& msui(mServerMsgUpdateMap[grpId]) ;
|
||||||
|
|
||||||
// (cyril) I'm removing this, because the msgUpdateTS is updated when new messages are received by calling locked_stampMsgServerUpdateTS().
|
// (cyril) I'm removing this, because the msgUpdateTS is updated when new messages are received by calling locked_stampMsgServerUpdateTS().
|
||||||
@ -3000,7 +3000,7 @@ void RsGxsNetService::locked_genReqGrpTransaction(NxsTransaction* tr)
|
|||||||
}
|
}
|
||||||
// FIXTESTS global variable rsReputations not available in unittests!
|
// FIXTESTS global variable rsReputations not available in unittests!
|
||||||
|
|
||||||
#warning Update the code below to correctly send/recv dependign on reputation
|
#warning csoler 2016-12-23: Update the code below to correctly send/recv dependign on reputation
|
||||||
if(!grpSyncItem->authorId.isNull() && mReputations->overallReputationLevel(grpSyncItem->authorId) == RsReputations::REPUTATION_LOCALLY_NEGATIVE)
|
if(!grpSyncItem->authorId.isNull() && mReputations->overallReputationLevel(grpSyncItem->authorId) == RsReputations::REPUTATION_LOCALLY_NEGATIVE)
|
||||||
{
|
{
|
||||||
#ifdef NXS_NET_DEBUG_0
|
#ifdef NXS_NET_DEBUG_0
|
||||||
|
@ -45,7 +45,7 @@ bool AuthorPending::getAuthorRep(GixsReputation& rep, const RsGxsId& authorId, c
|
|||||||
rep.id = authorId ;
|
rep.id = authorId ;
|
||||||
rep.reputation_level = mRep->overallReputationLevel(authorId);
|
rep.reputation_level = mRep->overallReputationLevel(authorId);
|
||||||
|
|
||||||
#warning can it happen that reputations do not have the info yet?
|
#warning csoler 2017-01-10: Can it happen that reputations do not have the info yet?
|
||||||
return true ;
|
return true ;
|
||||||
#ifdef TO_BE_REMOVED
|
#ifdef TO_BE_REMOVED
|
||||||
{
|
{
|
||||||
|
@ -627,7 +627,7 @@ bool getX509id(X509 *x509, RsPeerId& xid)
|
|||||||
* more randomness
|
* more randomness
|
||||||
*/
|
*/
|
||||||
|
|
||||||
#warning this is cryptographically horrible. We should do a hash of the public key here!!!
|
#warning csoler 2017-02-19: This is cryptographically horrible. We should do a hash of the public key here!!!
|
||||||
|
|
||||||
xid = RsPeerId(&signdata[signlen - CERTSIGNLEN]) ;
|
xid = RsPeerId(&signdata[signlen - CERTSIGNLEN]) ;
|
||||||
|
|
||||||
|
@ -1035,7 +1035,7 @@ bool p3GxsReputation::setOwnOpinion(const RsGxsId& gxsid, const RsReputations::O
|
|||||||
|
|
||||||
if (rit == mReputations.end())
|
if (rit == mReputations.end())
|
||||||
{
|
{
|
||||||
#warning we should set the owner node id here.
|
#warning csoler 2017-01-05: We should set the owner node id here.
|
||||||
mReputations[gxsid] = Reputation(gxsid);
|
mReputations[gxsid] = Reputation(gxsid);
|
||||||
rit = mReputations.find(gxsid);
|
rit = mReputations.find(gxsid);
|
||||||
}
|
}
|
||||||
|
@ -788,7 +788,7 @@ bool p3IdService::createIdentity(uint32_t& token, RsIdentityParameters ¶ms)
|
|||||||
|
|
||||||
if (params.isPgpLinked)
|
if (params.isPgpLinked)
|
||||||
{
|
{
|
||||||
#warning Backward compatibility issue to fix here in v0.7.0
|
#warning csoler 2017-02-07: Backward compatibility issue to fix here in v0.7.0
|
||||||
|
|
||||||
// This is a hack, because a bad decision led to having RSGXSID_GROUPFLAG_REALID be equal to GXS_SERV::FLAG_PRIVACY_PRIVATE.
|
// This is a hack, because a bad decision led to having RSGXSID_GROUPFLAG_REALID be equal to GXS_SERV::FLAG_PRIVACY_PRIVATE.
|
||||||
// In order to keep backward compatibility, we'll also add the new value
|
// In order to keep backward compatibility, we'll also add the new value
|
||||||
|
@ -1993,7 +1993,7 @@ QString IdDialog::createUsageString(const RsIdentityUsage& u) const
|
|||||||
return tr("Membership verification in circle %1.").arg(QString::fromStdString(u.mGrpId.toStdString()));
|
return tr("Membership verification in circle %1.").arg(QString::fromStdString(u.mGrpId.toStdString()));
|
||||||
}
|
}
|
||||||
|
|
||||||
#warning TODO! Add the different strings and translations here.
|
#warning TODO! csoler 2017-01-03: Add the different strings and translations here.
|
||||||
default:
|
default:
|
||||||
return QString("Undone yet");
|
return QString("Undone yet");
|
||||||
}
|
}
|
||||||
|
@ -77,7 +77,7 @@ class RetroShareLink
|
|||||||
RetroShareLink(const QUrl& url);
|
RetroShareLink(const QUrl& url);
|
||||||
RetroShareLink(const QString& url);
|
RetroShareLink(const QString& url);
|
||||||
|
|
||||||
#warning these methods should be static and return a created link
|
#warning csoler 2017-01-04: These methods should be static and return a created link
|
||||||
bool createFile(const QString& name, uint64_t size, const QString& hash);
|
bool createFile(const QString& name, uint64_t size, const QString& hash);
|
||||||
bool createExtraFile(const QString& name, uint64_t size, const QString& hash, const QString& ssl_id);
|
bool createExtraFile(const QString& name, uint64_t size, const QString& hash, const QString& ssl_id);
|
||||||
bool createPerson(const RsPgpId &id);
|
bool createPerson(const RsPgpId &id);
|
||||||
|
Loading…
x
Reference in New Issue
Block a user