Skip to content

Commit bf4713f

Browse files
committed
Added missing comments.
1 parent d6cdee3 commit bf4713f

File tree

2 files changed

+13
-0
lines changed

2 files changed

+13
-0
lines changed

Source/ExcelDna.Integration/Registration/ExcelRegistration.cs

+7
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,13 @@
88

99
namespace ExcelDna.Registration
1010
{
11+
// Ideas:
12+
// * Object Instances - Methods and Properties (with INotifyPropertyChanged support, and Disposable from Observable handles)
13+
// * Struct semantics, like built-in COMPLEX data
14+
// * Cache - PostSharp example from: http://vimeo.com/66549243 (esp. MethodExecutionTag for keeping stuff together)
15+
// * Apply a Module name XlQualifiedName(true), or use Class Name.
16+
17+
// A first attempt to allow chaining of the Registration rewrites.
1118
public static class ExcelRegistration
1219
{
1320
/// <summary>

Source/ExcelDna.Integration/Registration/ParameterConversionConfiguration.cs

+6
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,12 @@
55

66
namespace ExcelDna.Registration
77
{
8+
// Used for parameter and return type conversions (when these can be done without interfering with the rest of the function).
9+
10+
// CONSIDER: Add a name to the XXXConversion for tracing and debugging
11+
// CONSIDER: Do we need to consider Co-/Contravariance and allow processing of sub-/super-types?
12+
// What about native async function, they return 'void' type?
13+
814
public class ParameterConversionConfiguration
915
{
1016
internal class ParameterConversion

0 commit comments

Comments
 (0)