# HG changeset patch
# User Gregory Szorc <gregory.szorc@gmail.com>
# Date 1499363663 25200
# Node ID 1e9fd2c35ae36078dc60fd5282689b7949ab6fda
# Parent  74923cec6fdbceca769118d554a9fd7da37053b8
sparse: remove reference to simplecache

This is a 3rd party extension authored by Facebook. References in
core are not appropriate.

It will be possible to restore this code/optimization via
monkeypatching. So Facebook won't lose any functionality.

The removed code is important for performance. So add a comment
tracking it.

diff -r 74923cec6fdb -r 1e9fd2c35ae3 hgext/sparse.py
--- a/hgext/sparse.py	Sat Jul 01 10:24:31 2017 -0700
+++ b/hgext/sparse.py	Thu Jul 06 10:54:23 2017 -0700
@@ -436,16 +436,9 @@
             return includes, excludes, profiles
 
         def getrawprofile(self, profile, changeid):
-            try:
-                simplecache = extensions.find('simplecache')
-                node = self[changeid].hex()
-                def func():
-                    return self.filectx(profile, changeid=changeid).data()
-                key = 'sparseprofile:%s:%s' % (profile.replace('/', '__'), node)
-                return simplecache.memoize(func, key,
-                        simplecache.stringserializer, self.ui)
-            except KeyError:
-                return self.filectx(profile, changeid=changeid).data()
+            # TODO add some kind of cache here because this incurs a manifest
+            # resolve and can be slow.
+            return self.filectx(profile, changeid=changeid).data()
 
         def sparsechecksum(self, filepath):
             fh = open(filepath)