Added more comments and todos as per code review feedback

This commit is contained in:
Sheetal Nandi 2014-08-06 11:26:34 -07:00
parent 0e76a82072
commit e31aa9e12a

View File

@ -711,6 +711,20 @@ module ts {
return { accessibility: SymbolAccessibility.Accessible };
}
// TODO(shkamat): Handle static method of class
// If we havent got the accessible symbol doesnt mean the symbol is actually inaccessible.
// It could be qualified symbol and hence verify the path
// eg:
// module m {
// export class c {
// }
// }
// var x: typeof m.c
// In the above example when we start with checking if typeof m.c symbol is accessible,
// we are going to see if c can be accessed in scope directly.
// But it cant, hence the accessible is going to be undefined, but that doesnt mean m.c is accessible
// It is accessible if the parent m is accessible because then m.c can be accessed through qualification
meaningToLook = SymbolFlags.Namespace;
symbol = symbol.parent;
}