13
0
livetrax/gtk2_ardour/note_base.h
Tim Mayberry e7b8e98db2 Replace static PBD::Signal in NoteBase with direct call to MidiRegionView
NoteBaseDeleted signal is static so each MidiRegionView(MRV) gets notified
about the deletion of each NodeBase instance even if it is contained in another
MRV

The NoteBase and MRV classes are currently coupled anyway, so this change uses
the reference to the MRV parent to directly call the parent when the NoteBase
is deleted. This is all in the GUI thread so I'm not sure why a PBD::Signal was
being used?

If the MRV class is the only reference holder to the NoteBase class
then I'm not sure if a callback is needed, perhaps the MRV should just remove
the note from the selection before deleting it but I'm not that familiar with
the code.

Signal emission/calls static NoteBaseDeleted signal vs direct with 10540
NoteBase instances.

static:

After Load Session: 6360638
After Unload Session: 12221026(5860388)

direct:

After load Session: 10540
After unload Session: 21080

Session Load/Unload time in master, debug/release with ~10000 Notes(seconds)

Load Debug: 32, 26
Unload Debug: 83
Load Release 32, 20, 42
Unload Release 26, 25

Session Load/Unload time with direct call debug/release(seconds)

Load Debug: 21.7, 18.1
Unload Debug: 69.4, 71
Load Release: 22.6, 13.4, 17.7
Unload Release: 24, 23.5

This is not a large Session, 1500 regions, 10000 notes so there is probably
some other funky stuff going on that needs fixing.
2015-10-17 22:15:07 -04:00

156 lines
4.8 KiB
C++

/*
Copyright (C) 2007 Paul Davis
Author: David Robillard
This program is free software; you can redistribute it and/or modify
it under the terms of the GNU General Public License as published by
the Free Software Foundation; either version 2 of the License, or
(at your option) any later version.
This program is distributed in the hope that it will be useful,
but WITHOUT ANY WARRANTY; without even the implied warranty of
MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
GNU General Public License for more details.
You should have received a copy of the GNU General Public License
along with this program; if not, write to the Free Software
Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
*/
#ifndef __gtk_ardour_note_base_h__
#define __gtk_ardour_note_base_h__
#include <boost/shared_ptr.hpp>
#include "canvas/types.h"
#include "evoral/types.hpp"
#include "rgb_macros.h"
#include "ui_config.h"
class Editor;
class MidiRegionView;
namespace Evoral {
template<typename T> class Note;
}
namespace ArdourCanvas {
class Item;
class Text;
}
/** Base class for canvas notes (sustained note rectangles and hit diamonds).
*
* This is not actually a canvas item itself to avoid the dreaded diamond
* inheritance pattern, since various types of canvas items (Note (rect), Hit
* (diamond), etc) need to share this functionality but can't share an
* ancestor.
*
* Note: Because of this, derived classes need to manually bounce events to
* on_event, it won't happen automatically.
*/
class NoteBase : public sigc::trackable
{
public:
typedef Evoral::Note<Evoral::Beats> NoteType;
NoteBase (MidiRegionView& region, bool, const boost::shared_ptr<NoteType> note = boost::shared_ptr<NoteType>());
virtual ~NoteBase ();
void set_item (ArdourCanvas::Item *);
ArdourCanvas::Item* item() const { return _item; }
virtual void show() = 0;
virtual void hide() = 0;
bool valid() const { return _valid; }
void invalidate ();
void validate ();
bool selected() const { return _selected; }
void set_selected(bool yn);
virtual void move_event(double dx, double dy) = 0;
uint32_t base_color();
void show_velocity();
void hide_velocity();
/** Channel changed for this specific event */
void on_channel_change(uint8_t channel);
/** Channel selection changed */
void on_channel_selection_change(uint16_t selection);
virtual void set_outline_color(uint32_t c) = 0;
virtual void set_fill_color(uint32_t c) = 0;
virtual void set_ignore_events(bool ignore) = 0;
virtual ArdourCanvas::Coord x0 () const = 0;
virtual ArdourCanvas::Coord y0 () const = 0;
virtual ArdourCanvas::Coord x1 () const = 0;
virtual ArdourCanvas::Coord y1 () const = 0;
float mouse_x_fraction() const { return _mouse_x_fraction; }
float mouse_y_fraction() const { return _mouse_y_fraction; }
const boost::shared_ptr<NoteType> note() const { return _note; }
MidiRegionView& region_view() const { return _region; }
inline static uint32_t meter_style_fill_color(uint8_t vel, bool selected) {
if (selected) {
return UIConfiguration::instance().color_mod ("midi note selected", "midi note");
} else if (vel < 64) {
return UINT_INTERPOLATE(
UIConfiguration::instance().color_mod ("midi note min", "midi note"),
UIConfiguration::instance().color_mod ("midi note mid", "midi note"),
(vel / (double)63.0));
} else {
return UINT_INTERPOLATE(
UIConfiguration::instance().color_mod ("midi note mid", "midi note"),
UIConfiguration::instance().color_mod ("midi note max", "midi note"),
((vel-64) / (double)63.0));
}
}
/// calculate outline colors from fill colors of notes
inline static uint32_t calculate_outline(uint32_t color, bool selected=false) {
if (selected) {
return UIConfiguration::instance().color ("midi note selected outline");
} else {
return UINT_INTERPOLATE(color, 0x000000ff, 0.5);
}
}
/// hue circle divided into 16 equal-looking parts, courtesy Thorsten Wilms
static const uint32_t midi_channel_colors[16];
bool mouse_near_ends () const;
virtual bool big_enough_to_trim () const;
protected:
enum State { None, Pressed, Dragging };
MidiRegionView& _region;
ArdourCanvas::Item* _item;
ArdourCanvas::Text* _text;
State _state;
const boost::shared_ptr<NoteType> _note;
bool _with_events;
bool _own_note;
bool _selected;
bool _valid;
float _mouse_x_fraction;
float _mouse_y_fraction;
void set_mouse_fractions (GdkEvent*);
private:
bool event_handler (GdkEvent *);
};
#endif /* __gtk_ardour_note_h__ */