Navigation Menu

Skip to content
This repository has been archived by the owner on Jan 15, 2021. It is now read-only.

Commit

Permalink
Fixed #287 - Infrastructure is now in place
Browse files Browse the repository at this point in the history
  • Loading branch information
avanderhoorn committed Mar 18, 2013
1 parent 40279bc commit 4cf8977
Show file tree
Hide file tree
Showing 3 changed files with 33 additions and 0 deletions.
2 changes: 2 additions & 0 deletions source/Glimpse.Core/Glimpse.Core.csproj
Expand Up @@ -102,6 +102,8 @@
<Compile Include="Extensibility\RuntimePolicy.cs" />
<Compile Include="Extensibility\Subscriber.cs" />
<Compile Include="Extensions\EnumExtensions.cs" />
<Compile Include="Setting\Initializer.cs" />
<Compile Include="System.cs" />
<Compile Include="Message\ISourceMessage.cs" />
<Compile Include="Message\ITraceMessage.cs" />
<Compile Include="Message\TimelineCategory.cs" />
Expand Down
13 changes: 13 additions & 0 deletions source/Glimpse.Core/Setting/Initializer.cs
@@ -0,0 +1,13 @@
namespace Glimpse.Core.Setting
{
/// <summary>
/// Designed solely for plugin authors to add Initialize options
/// to via extension methods.
/// </summary>
public class Initializer
{
internal Initializer()
{
}
}
}
18 changes: 18 additions & 0 deletions source/Glimpse.Core/System.cs
@@ -0,0 +1,18 @@
using Glimpse.Core.Setting;

namespace Glimpse
{
/// <summary>
/// Hook that is used to hang any system settings
/// </summary>
public static class Settings
{
/// <summary>
/// Hook that is used to hang initialization code from. No method
/// here is required ot be called. You should only need to use this
/// code if Glimpse doesn't initialize soon enough in the application
/// life cycle for your purposes.
/// </summary>
public static Initializer Initialize = new Initializer();
}
}

1 comment on commit 4cf8977

@nikmd23
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this whole commit could be done with just one class.

It seems like this class should be in Glimpse.Core.Extensibility as well - if it is meant as an extensibility point.

Please sign in to comment.