fix issue with invalid (up|down)total($|kb) calculation (see #127)

also renamed *total* tags to *tot* for consistency with other tags:

  * uptotal.* -> uptot.*
  * downtotal.* -> downtot.*
  * pstotal -> pstot
This commit is contained in:
Evgenii Alekseev 2017-07-24 15:07:36 +03:00
parent d2e6f2fe38
commit 5e08434835
7 changed files with 23 additions and 20 deletions

View File

@ -147,8 +147,8 @@ QStringList AWKeyCache::getRequiredKeys(const QStringList &_keys,
used << "swapgb" used << "swapgb"
<< "swapfreegb"; << "swapfreegb";
// network keys // network keys
QStringList netKeys({"up", "upkb", "uptotal", "uptotalkb", "upunits", QStringList netKeys({"up", "upkb", "uptot", "uptotkb", "upunits",
"down", "downkb", "downtotal", "downtotalkb", "down", "downkb", "downtot", "downtotkb",
"downunits"}); "downunits"});
for (auto &key : netKeys) { for (auto &key : netKeys) {
if (!used.contains(key)) if (!used.contains(key))

View File

@ -122,12 +122,12 @@ QStringList AWKeyOperations::dictKeys() const
for (int i = 0; i < m_devices["net"].count(); i++) { for (int i = 0; i < m_devices["net"].count(); i++) {
allKeys.append(QString("downunits%1").arg(i)); allKeys.append(QString("downunits%1").arg(i));
allKeys.append(QString("upunits%1").arg(i)); allKeys.append(QString("upunits%1").arg(i));
allKeys.append(QString("downtotalkb%1").arg(i)); allKeys.append(QString("downtotkb%1").arg(i));
allKeys.append(QString("downtotal%1").arg(i)); allKeys.append(QString("downtot%1").arg(i));
allKeys.append(QString("downkb%1").arg(i)); allKeys.append(QString("downkb%1").arg(i));
allKeys.append(QString("down%1").arg(i)); allKeys.append(QString("down%1").arg(i));
allKeys.append(QString("uptotalkb%1").arg(i)); allKeys.append(QString("uptotkb%1").arg(i));
allKeys.append(QString("uptotal%1").arg(i)); allKeys.append(QString("uptot%1").arg(i));
allKeys.append(QString("upkb%1").arg(i)); allKeys.append(QString("upkb%1").arg(i));
allKeys.append(QString("up%1").arg(i)); allKeys.append(QString("up%1").arg(i));
} }

View File

@ -291,13 +291,13 @@ void AWKeys::calculateValues()
= m_keyOperator->devices("net").indexOf(m_values["netdev"].toString()); = m_keyOperator->devices("net").indexOf(m_values["netdev"].toString());
m_values["down"] = m_values[QString("down%1").arg(netIndex)]; m_values["down"] = m_values[QString("down%1").arg(netIndex)];
m_values["downkb"] = m_values[QString("downkb%1").arg(netIndex)]; m_values["downkb"] = m_values[QString("downkb%1").arg(netIndex)];
m_values["downtotal"] = m_values[QString("downtotal%1").arg(netIndex)]; m_values["downtot"] = m_values[QString("downtot%1").arg(netIndex)];
m_values["downtotalkb"] = m_values[QString("downtotalkb%1").arg(netIndex)]; m_values["downtotkb"] = m_values[QString("downtotkb%1").arg(netIndex)];
m_values["downunits"] = m_values[QString("downunits%1").arg(netIndex)]; m_values["downunits"] = m_values[QString("downunits%1").arg(netIndex)];
m_values["up"] = m_values[QString("up%1").arg(netIndex)]; m_values["up"] = m_values[QString("up%1").arg(netIndex)];
m_values["upkb"] = m_values[QString("upkb%1").arg(netIndex)]; m_values["upkb"] = m_values[QString("upkb%1").arg(netIndex)];
m_values["uptotal"] = m_values[QString("uptotal%1").arg(netIndex)]; m_values["uptot"] = m_values[QString("uptot%1").arg(netIndex)];
m_values["uptotalkb"] = m_values[QString("uptotalkb%1").arg(netIndex)]; m_values["uptotkb"] = m_values[QString("uptotkb%1").arg(netIndex)];
m_values["upunits"] = m_values[QString("upunits%1").arg(netIndex)]; m_values["upunits"] = m_values[QString("upunits%1").arg(netIndex)];
// swaptot* // swaptot*

View File

@ -46,9 +46,13 @@ AWKeysAggregator::AWKeysAggregator(QObject *_parent)
// network // network
m_formatter["down"] = FormatterType::NetSmartFormat; m_formatter["down"] = FormatterType::NetSmartFormat;
m_formatter["downkb"] = FormatterType::Integer; m_formatter["downkb"] = FormatterType::Integer;
m_formatter["downtot"] = FormatterType::MemMBFormat;
m_formatter["downtotkb"] = FormatterType::Integer;
m_formatter["downunits"] = FormatterType::NetSmartUnits; m_formatter["downunits"] = FormatterType::NetSmartUnits;
m_formatter["up"] = FormatterType::NetSmartFormat; m_formatter["up"] = FormatterType::NetSmartFormat;
m_formatter["upkb"] = FormatterType::Integer; m_formatter["upkb"] = FormatterType::Integer;
m_formatter["uptot"] = FormatterType::MemMBFormat;
m_formatter["uptotkb"] = FormatterType::Integer;
m_formatter["upunits"] = FormatterType::NetSmartUnits; m_formatter["upunits"] = FormatterType::NetSmartUnits;
// swap // swap
m_formatter["swap"] = FormatterType::Float; m_formatter["swap"] = FormatterType::Float;
@ -468,11 +472,11 @@ QStringList AWKeysAggregator::registerSource(const QString &_source,
int index = m_devices["net"].indexOf(_source.split('/')[2]); int index = m_devices["net"].indexOf(_source.split('/')[2]);
if (index > -1) { if (index > -1) {
// kb // kb
QString key = QString("%1totalkb%2").arg(type).arg(index); QString key = QString("%1totkb%2").arg(type).arg(index);
m_map[_source] = key; m_map[_source] = key;
m_formatter[key] = FormatterType::Integer; m_formatter[key] = FormatterType::Integer;
// mb // mb
key = QString("%1total%2").arg(type).arg(index); key = QString("%1tot%2").arg(type).arg(index);
m_map.insertMulti(_source, key); m_map.insertMulti(_source, key);
m_formatter[key] = FormatterType::MemMBFormat; m_formatter[key] = FormatterType::MemMBFormat;
} }
@ -498,8 +502,8 @@ QStringList AWKeysAggregator::registerSource(const QString &_source,
m_formatter["ps"] = FormatterType::List; m_formatter["ps"] = FormatterType::List;
} else if (_source == "ps/total/count") { } else if (_source == "ps/total/count") {
// total processes count // total processes count
m_map[_source] = "pstotal"; m_map[_source] = "pstot";
m_formatter["pstotal"] = FormatterType::NoFormat; m_formatter["pstot"] = FormatterType::NoFormat;
} else if (_source.startsWith("quotes")) { } else if (_source.startsWith("quotes")) {
// quotes // quotes
QString key = _source; QString key = _source;

View File

@ -24,7 +24,6 @@
#include "version.h" #include "version.h"
class AWFormatterHelper; class AWFormatterHelper;
class AWKeysAggregator : public QObject class AWKeysAggregator : public QObject

View File

@ -121,7 +121,7 @@ queueLimit=0
swapTooltip=true swapTooltip=true
swapTooltipColor=#ffff00 swapTooltipColor=#ffff00
tempUnits=Celsius tempUnits=Celsius
text="<body bgcolor=\"#000000\">\n<p align=\"justify\">Uptime: $cuptime<br>\nRAM: &nbsp;$mem&nbsp;&nbsp;$bar5<br>\nSwap: $swap&nbsp;&nbsp;$bar6<br>\nCPU: &nbsp;$cpu&nbsp;&nbsp;$bar7<br>\nCPU Temp: $temp0&deg;C<br>\nDown: $down$downunits&nbsp;&nbsp;&nbsp;&nbsp;$downtotal<br>\n$bar8<br>\nUp:&nbsp;&nbsp; $up$upunits&nbsp;&nbsp;&nbsp;&nbsp;$uptotal<br>\n$bar9<br></p>\n</body>\n" text="<body bgcolor=\"#000000\">\n<p align=\"justify\">Uptime: $cuptime<br>\nRAM: &nbsp;$mem&nbsp;&nbsp;$bar5<br>\nSwap: $swap&nbsp;&nbsp;$bar6<br>\nCPU: &nbsp;$cpu&nbsp;&nbsp;$bar7<br>\nCPU Temp: $temp0&deg;C<br>\nDown: $down$downunits&nbsp;&nbsp;&nbsp;&nbsp;$downtot<br>\n$bar8<br>\nUp:&nbsp;&nbsp; $up$upunits&nbsp;&nbsp;&nbsp;&nbsp;$uptot<br>\n$bar9<br></p>\n</body>\n"
textAlign=center textAlign=center
tooltipBackground=#ffffff tooltipBackground=#ffffff
tooltipNumber=100 tooltipNumber=100

View File

@ -57,10 +57,10 @@ const char STATIC_KEYS[]
= "time,isotime,shorttime,longtime,tstime,ctime,uptime,cuptime,cpucl,cpu," = "time,isotime,shorttime,longtime,tstime,ctime,uptime,cuptime,cpucl,cpu,"
"gputemp,gpu,memmb,memgb,memfreemb,memfreegb,memtotmb,memtotgb,memusedmb," "gputemp,gpu,memmb,memgb,memfreemb,memfreegb,memtotmb,memtotgb,memusedmb,"
"memusedgb,mem,swapmb,swapgb,swapfreemb,swapfreegb,swaptotmb,swaptotgb," "memusedgb,mem,swapmb,swapgb,swapfreemb,swapfreegb,swaptotmb,swaptotgb,"
"swap,downunits,upunits,downkb,downtotalkb,downtotal,down,uptotalkb," "swap,downunits,upunits,downkb,downtotkb,downtot,down,uptotkb,uptot,upkb,"
"uptotal,upkb,up,netdev,ac,bat,album,artist,duration,progress,title," "up,netdev,ac,bat,album,artist,duration,progress,title,dalbum,dartist,"
"dalbum,dartist,dtitle,salbum,sartist,stitle,pscount,pstotal,ps,desktop," "dtitle,salbum,sartist,stitle,pscount,pstot,ps,desktop,ndesktop,"
"ndesktop,tdesktops,la15,la5,la1"; "tdesktops,la15,la5,la1";
#cmakedefine BUILD_FUTURE #cmakedefine BUILD_FUTURE
#cmakedefine BUILD_LOAD #cmakedefine BUILD_LOAD
#cmakedefine BUILD_TESTING #cmakedefine BUILD_TESTING