annotate layer/LogColourScale.cpp @ 854:c17719e488c9

Fix some potential null-pointer derefs, and simplify some logic where loops were used with an unconditional "break" that meant they could only happen once (from coverity scan)
author Chris Cannam
date Wed, 03 Sep 2014 12:04:22 +0100
parents 1d526ba11a24
children 3ca3b8fbbcee
rev   line source
Chris@699 1 /* -*- c-basic-offset: 4 indent-tabs-mode: nil -*- vi:set ts=8 sts=4 sw=4: */
Chris@699 2
Chris@699 3 /*
Chris@699 4 Sonic Visualiser
Chris@699 5 An audio file viewer and annotation editor.
Chris@699 6 Centre for Digital Music, Queen Mary, University of London.
Chris@699 7 This file copyright 2006-2013 Chris Cannam and QMUL.
Chris@699 8
Chris@699 9 This program is free software; you can redistribute it and/or
Chris@699 10 modify it under the terms of the GNU General Public License as
Chris@699 11 published by the Free Software Foundation; either version 2 of the
Chris@699 12 License, or (at your option) any later version. See the file
Chris@699 13 COPYING included with this distribution for more information.
Chris@699 14 */
Chris@699 15
Chris@699 16 #include "LogColourScale.h"
Chris@699 17 #include "ColourScaleLayer.h"
Chris@699 18
Chris@699 19 #include "base/LogRange.h"
Chris@699 20
Chris@699 21 #include <QPainter>
Chris@699 22
Chris@699 23 #include <cmath>
Chris@699 24
Chris@699 25 #include "view/View.h"
Chris@699 26
Chris@699 27 int
Chris@805 28 LogColourScale::getWidth(View *,
Chris@699 29 QPainter &paint)
Chris@699 30 {
Chris@699 31 return paint.fontMetrics().width("-000.00") + 15;
Chris@699 32 }
Chris@699 33
Chris@699 34 void
Chris@699 35 LogColourScale::paintVertical(View *v,
Chris@699 36 const ColourScaleLayer *layer,
Chris@699 37 QPainter &paint,
Chris@805 38 int /* x0 */,
Chris@699 39 float minlog,
Chris@699 40 float maxlog)
Chris@699 41 {
Chris@699 42 int h = v->height();
Chris@699 43
Chris@699 44 int n = 10;
Chris@699 45
Chris@699 46 float val = minlog;
Chris@699 47 float inc = (maxlog - val) / n;
Chris@699 48
Chris@699 49 char buffer[40];
Chris@699 50
Chris@699 51 int boxx = 5, boxy = 5;
Chris@699 52 if (layer->getScaleUnits() != "") {
Chris@699 53 boxy += paint.fontMetrics().height();
Chris@699 54 }
Chris@699 55 int boxw = 10, boxh = h - boxy - 5;
Chris@699 56
Chris@699 57 int tx = 5 + boxx + boxw;
Chris@699 58 paint.drawRect(boxx, boxy, boxw, boxh);
Chris@699 59
Chris@699 60 paint.save();
Chris@699 61 for (int y = 0; y < boxh; ++y) {
Chris@699 62 float val = ((boxh - y) * (maxlog - minlog)) / boxh + minlog;
Chris@699 63 paint.setPen(layer->getColourForValue(v, LogRange::unmap(val)));
Chris@699 64 paint.drawLine(boxx + 1, y + boxy + 1, boxx + boxw, y + boxy + 1);
Chris@699 65 }
Chris@699 66 paint.restore();
Chris@699 67
Chris@699 68 int dp = 0;
Chris@699 69 if (inc > 0) {
Chris@699 70 int prec = trunc(log10f(inc));
Chris@699 71 prec -= 1;
Chris@699 72 if (prec < 0) dp = -prec;
Chris@699 73 }
Chris@699 74
Chris@699 75 for (int i = 0; i < n; ++i) {
Chris@699 76
Chris@699 77 int y, ty;
Chris@699 78
Chris@699 79 y = boxy + int(boxh - ((val - minlog) * boxh) / (maxlog - minlog));
Chris@699 80
Chris@699 81 ty = y - paint.fontMetrics().height() +
Chris@699 82 paint.fontMetrics().ascent() + 2;
Chris@699 83
Chris@699 84 double dv = LogRange::unmap(val);
Chris@699 85 int digits = trunc(log10f(dv));
Chris@699 86 int sf = dp + (digits > 0 ? digits : 0);
Chris@699 87 if (sf < 2) sf = 2;
Chris@699 88 sprintf(buffer, "%.*g", sf, dv);
Chris@699 89
Chris@699 90 QString label = QString(buffer);
Chris@699 91
Chris@699 92 paint.drawLine(boxx + boxw - boxw/3, y, boxx + boxw, y);
Chris@699 93 paint.drawText(tx, ty, label);
Chris@699 94
Chris@699 95 val += inc;
Chris@699 96 }
Chris@699 97 }