i18n: Memoize dcnpgettext (#101)

* i18n: Memoize dcnpgettext * i18n: Export dcnpgettext inline
WordPress · Apr 12, 2018 · 2bd43bb · 2bd43bb
1 parent ed56276
commit 2bd43bb
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 2 deletions.
diff --git a/packages/i18n/benchmark/index.js b/packages/i18n/benchmark/index.js
@@ -0,0 +1,11 @@
+const Benchmark = require( 'benchmark' );
+const { __ } = require( '../' );
+
+const suite = new Benchmark.Suite;
+
+suite
+	.add( '__', () => {
+		__( 'Translate' );
+	} )
+	.on( 'cycle', ( event ) => console.log( event.target.toString() ) )
+	.run( { async: true } );
diff --git a/packages/i18n/src/index.js b/packages/i18n/src/index.js
@@ -69,15 +69,15 @@ export function getI18n() {
  *
  * @return {string} The translated string.
  */
-export function dcnpgettext( domain = 'default', context, single, plural, number ) {
+export const dcnpgettext = memoize( ( domain = 'default', context, single, plural, number ) => {
 	try {
 		return getI18n().dcnpgettext( domain, context, single, plural, number );
 	} catch ( error ) {
 		logErrorOnce( 'Jed localization error: \n\n' + error.toString() );
 
 		return single;
 	}
-}
+} );
 
 /**
  * Retrieve the translation of text.