Re: [Banshee-List] banshee r2978 - in trunk/banshee: . src/Core/Banshee.Services/Banshee.Collection.Database src/Core/Banshee.Services/Banshee.Database src/Core/Hyena src/Core/Hyena/Hyena.Data src/Core/Hyena/Hyena.Data.Sqlite

["Gabriel Burt" <gabriel burt gmail com>]

On Jan 17, 2008 12:13 AM,  <scottp svn gnome org> wrote:
> --- trunk/banshee/src/Core/Banshee.Services/Banshee.Database/BansheeDatabaseModelCache.cs       (original)
> +++ trunk/banshee/src/Core/Banshee.Services/Banshee.Database/BansheeDatabaseModelCache.cs       Thu Jan 17 06:13:51 2008
> @@ -40,15 +40,13 @@
>          public BansheeDatabaseModelCache (HyenaSqliteConnection connection, string uuid, ICacheableDatabaseModel<T> model) : base (connection, uuid, model)
>          {
>          }
> -
> -        private const string model_table = "CoreCacheModels";
> +
>          protected override string CacheModelsTableName {
> -            get { return model_table; }
> +            get { return "CoreCacheModels"; }
>          }
>
> -        private const string cache_table = "CoreCache";
>          protected override string CacheTableName {
> -            get { return cache_table; }
> +            get { return "CoreCache"; }
>          }
>      }
>  }

Nice, for some reason I thought using the const would be better, but
looks like it produces identical IL (just a ldstr and ret), except
with the const it also has to declare the static variables which are
then not used.

Gabriel