GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — perf(common): use `ngDevMode` to tree-shake warnings (#39964) · angular/angular@bf3de9b

File tree

Expand file treeCollapse file tree
Open diff view settings
Filter options
  • packages/common/src
    • directives
      • ng_for_of.ts
    • pipes
      • number_pipe.ts
Expand file treeCollapse file tree
Open diff view settings
Collapse file

‎packages/common/src/directives/ng_for_of.ts‎

Copy file name to clipboardExpand all lines: packages/common/src/directives/ng_for_of.ts
+2-2Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
* found in the LICENSE file at https://angular.io/license
77
*/
88

9-
import {Directive, DoCheck, EmbeddedViewRef, Input, isDevMode, IterableChangeRecord, IterableChanges, IterableDiffer, IterableDiffers, NgIterable, TemplateRef, TrackByFunction, ViewContainerRef} from '@angular/core';
9+
import {Directive, DoCheck, EmbeddedViewRef, Input, IterableChangeRecord, IterableChanges, IterableDiffer, IterableDiffers, NgIterable, TemplateRef, TrackByFunction, ViewContainerRef} from '@angular/core';
1010

1111
/**
1212
* @publicApi
@@ -159,7 +159,7 @@ export class NgForOf<T, U extends NgIterable<T> = NgIterable<T>> implements DoCh
159159
*/
160160
@Input()
161161
set ngForTrackBy(fn: TrackByFunction<T>) {
162-
if (isDevMode() && fn != null && typeof fn !== 'function') {
162+
if ((typeof ngDevMode === 'undefined' || ngDevMode) && fn != null && typeof fn !== 'function') {
163163
// TODO(vicb): use a log service once there is a public one available
164164
if (<any>console && <any>console.warn) {
165165
console.warn(
Collapse file

‎packages/common/src/pipes/number_pipe.ts‎

Copy file name to clipboardExpand all lines: packages/common/src/pipes/number_pipe.ts
+1-1Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -241,7 +241,7 @@ export class CurrencyPipe implements PipeTransform {
241241
locale = locale || this._locale;
242242

243243
if (typeof display === 'boolean') {
244-
if (<any>console && <any>console.warn) {
244+
if ((typeof ngDevMode === 'undefined' || ngDevMode) && <any>console && <any>console.warn) {
245245
console.warn(
246246
`Warning: the currency pipe has been changed in Angular v5. The symbolDisplay option (third parameter) is now a string instead of a boolean. The accepted values are "code", "symbol" or "symbol-narrow".`);
247247
}

0 commit comments

Comments
0 (0)
You are visitor 01402520 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.