Skip to content

Commit fe6cab7

Browse files
committed
Disallow the non-generic version of Registry.as
We should always require an explicit type for this
1 parent f94cea3 commit fe6cab7

39 files changed

Lines changed: 64 additions & 65 deletions

File tree

src/vs/editor/common/config/commonEditorConfig.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -166,7 +166,7 @@ export abstract class CommonEditorConfiguration extends Disposable implements ed
166166

167167
}
168168

169-
const configurationRegistry = <IConfigurationRegistry>Registry.as(Extensions.Configuration);
169+
const configurationRegistry = Registry.as<IConfigurationRegistry>(Extensions.Configuration);
170170
const editorConfiguration: IConfigurationNode = {
171171
'id': 'editor',
172172
'order': 5,

src/vs/editor/standalone/browser/standaloneThemeServiceImpl.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ const VS_THEME_NAME = 'vs';
1919
const VS_DARK_THEME_NAME = 'vs-dark';
2020
const HC_BLACK_THEME_NAME = 'hc-black';
2121

22-
const colorRegistry = <IColorRegistry>Registry.as(Extensions.ColorContribution);
22+
const colorRegistry = Registry.as<IColorRegistry>(Extensions.ColorContribution);
2323
const themingRegistry = Registry.as<IThemingRegistry>(ThemingExtensions.ThemingContribution);
2424

2525
class StandaloneTheme implements IStandaloneTheme {

src/vs/platform/configuration/test/node/configurationService.test.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -132,7 +132,7 @@ suite('ConfigurationService - Node', () => {
132132
};
133133
}
134134

135-
const configurationRegistry = <IConfigurationRegistry>Registry.as(ConfigurationExtensions.Configuration);
135+
const configurationRegistry = Registry.as<IConfigurationRegistry>(ConfigurationExtensions.Configuration);
136136
configurationRegistry.registerConfiguration({
137137
'id': '_test',
138138
'type': 'object',
@@ -177,7 +177,7 @@ suite('ConfigurationService - Node', () => {
177177
});
178178

179179
test('lookup', () => {
180-
const configurationRegistry = <IConfigurationRegistry>Registry.as(ConfigurationExtensions.Configuration);
180+
const configurationRegistry = Registry.as<IConfigurationRegistry>(ConfigurationExtensions.Configuration);
181181
configurationRegistry.registerConfiguration({
182182
'id': '_test',
183183
'type': 'object',
@@ -218,7 +218,7 @@ suite('ConfigurationService - Node', () => {
218218
});
219219

220220
test('lookup with null', () => {
221-
const configurationRegistry = <IConfigurationRegistry>Registry.as(ConfigurationExtensions.Configuration);
221+
const configurationRegistry = Registry.as<IConfigurationRegistry>(ConfigurationExtensions.Configuration);
222222
configurationRegistry.registerConfiguration({
223223
'id': '_testNull',
224224
'type': 'object',

src/vs/platform/registry/common/platform.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,6 @@ export interface IRegistry {
2727
* Returns the extension functions and properties defined by the specified key or null.
2828
* @param id an extension identifier
2929
*/
30-
as(id: string): any;
3130
as<T>(id: string): T;
3231
}
3332

src/vs/platform/registry/test/common/platform.test.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -21,8 +21,8 @@ suite('Platform / Registry', () => {
2121
Platform.Registry.add('foo', { bar: true });
2222

2323
assert.ok(Platform.Registry.knows('foo'));
24-
assert.ok(Platform.Registry.as('foo').bar);
25-
assert.equal(Platform.Registry.as('foo').bar, true);
24+
assert.ok(Platform.Registry.as<any>('foo').bar);
25+
assert.equal(Platform.Registry.as<any>('foo').bar, true);
2626
});
2727

2828
test('registry - knows, as', function () {

src/vs/platform/update/node/update.config.contribution.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ import product from 'vs/platform/node/product';
1010
import { Registry } from 'vs/platform/registry/common/platform';
1111
import { IConfigurationRegistry, Extensions as ConfigurationExtensions } from 'vs/platform/configuration/common/configurationRegistry';
1212

13-
const configurationRegistry = <IConfigurationRegistry>Registry.as(ConfigurationExtensions.Configuration);
13+
const configurationRegistry = Registry.as<IConfigurationRegistry>(ConfigurationExtensions.Configuration);
1414
configurationRegistry.registerConfiguration({
1515
'id': 'update',
1616
'order': 15,

src/vs/workbench/browser/parts/quickopen/quickopen.contribution.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@ KeybindingsRegistry.registerCommandAndKeybindingRule({
4646
}
4747
});
4848

49-
const registry = <IWorkbenchActionRegistry>Registry.as(ActionExtensions.WorkbenchActions);
49+
const registry = Registry.as<IWorkbenchActionRegistry>(ActionExtensions.WorkbenchActions);
5050

5151
const globalQuickOpenKeybinding = { primary: KeyMod.CtrlCmd | KeyCode.KEY_P, secondary: [KeyMod.CtrlCmd | KeyCode.KEY_E], mac: { primary: KeyMod.CtrlCmd | KeyCode.KEY_P, secondary: null } };
5252

src/vs/workbench/parts/codeEditor/electron-browser/languageConfiguration/languageConfigurationExtensionPoint.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -581,5 +581,5 @@ const schema: IJSONSchema = {
581581

582582
}
583583
};
584-
let schemaRegistry = <IJSONContributionRegistry>Registry.as(Extensions.JSONContribution);
584+
let schemaRegistry = Registry.as<IJSONContributionRegistry>(Extensions.JSONContribution);
585585
schemaRegistry.registerSchema(schemaId, schema);

src/vs/workbench/parts/debug/electron-browser/debug.contribution.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -151,7 +151,7 @@ registry.registerWorkbenchAction(new SyncActionDescriptor(FocusBreakpointsViewAc
151151

152152

153153
// Register Quick Open
154-
(<IQuickOpenRegistry>Registry.as(QuickOpenExtensions.Quickopen)).registerQuickOpenHandler(
154+
(Registry.as<IQuickOpenRegistry>(QuickOpenExtensions.Quickopen)).registerQuickOpenHandler(
155155
new QuickOpenHandlerDescriptor(
156156
DebugQuickOpenHandler,
157157
DebugQuickOpenHandler.ID,

src/vs/workbench/parts/debug/electron-browser/debugConfigurationManager.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -215,7 +215,7 @@ const schema: IJSONSchema = {
215215
}
216216
};
217217

218-
const jsonRegistry = <IJSONContributionRegistry>Registry.as(JSONExtensions.JSONContribution);
218+
const jsonRegistry = Registry.as<IJSONContributionRegistry>(JSONExtensions.JSONContribution);
219219
jsonRegistry.registerSchema(launchSchemaId, schema);
220220
const DEBUG_SELECTED_CONFIG_NAME_KEY = 'debug.selectedconfigname';
221221
const DEBUG_SELECTED_ROOT = 'debug.selectedroot';

0 commit comments

Comments
 (0)