[banshee] ThickClient: prever ANE over NRE
- From: AndrÃs Aragoneses <aaragoneses src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [banshee] ThickClient: prever ANE over NRE
- Date: Wed, 18 Jan 2012 23:16:24 +0000 (UTC)
commit 4dc7c3a625d2216dc358d2c82926869c8b724a9f
Author: Andres G. Aragoneses <knocte gmail com>
Date: Wed Jan 18 23:19:10 2012 +0000
ThickClient: prever ANE over NRE
A user has managed to reproduce a NRE which
has a stacktrace that leads me to think that
the 'model' argument was null, so throw an
ArgumentNullException to be sure and get
better debugging info next time. (Will help
bgo#667625 if it appears again.)
Signed-off-by: Alexander Kojevnikov <alexk gnome org>
.../Banshee.Sources.Gui/SourceView.cs | 4 ++++
1 files changed, 4 insertions(+), 0 deletions(-)
---
diff --git a/src/Core/Banshee.ThickClient/Banshee.Sources.Gui/SourceView.cs b/src/Core/Banshee.ThickClient/Banshee.Sources.Gui/SourceView.cs
index a91d056..1401fa6 100644
--- a/src/Core/Banshee.ThickClient/Banshee.Sources.Gui/SourceView.cs
+++ b/src/Core/Banshee.ThickClient/Banshee.Sources.Gui/SourceView.cs
@@ -130,6 +130,10 @@ namespace Banshee.Sources.Gui
source_column.PackStart (header_renderer, true);
source_column.SetCellDataFunc (header_renderer, new Gtk.CellLayoutDataFunc ((layout, cell, model, iter) => {
+ if (model == null) {
+ throw new ArgumentNullException ("model");
+ }
+
var type = (SourceModel.EntryType) model.GetValue (iter, (int)SourceModel.Columns.Type);
header_renderer.Visible = type == SourceModel.EntryType.Group;
source_renderer.Visible = type == SourceModel.EntryType.Source;
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]