diff options
author | jthorn <jthorn@f88db872-0e4f-0410-b76b-b9085cfa78c5> | 2003-03-20 13:45:50 +0000 |
---|---|---|
committer | jthorn <jthorn@f88db872-0e4f-0410-b76b-b9085cfa78c5> | 2003-03-20 13:45:50 +0000 |
commit | 7f11670b08fc322118f0625532e3a15d7e471f02 (patch) | |
tree | 86e3ebf404883c2558c399976581331a9225cea0 | |
parent | 5582906aab7f861d6747c5db14a4d5796511364f (diff) |
slightly expand FIXME comments for patch_system::plus_or_minus_xyz_patch()
about slow vs fast implementation
git-svn-id: http://svn.einsteintoolkit.org/cactus/EinsteinAnalysis/AHFinderDirect/trunk@998 f88db872-0e4f-0410-b76b-b9085cfa78c5
-rw-r--r-- | src/patch/patch_system.cc | 7 | ||||
-rw-r--r-- | src/patch/patch_system.hh | 1 |
2 files changed, 5 insertions, 3 deletions
diff --git a/src/patch/patch_system.cc b/src/patch/patch_system.cc index 10cbd5b..cea3544 100644 --- a/src/patch/patch_system.cc +++ b/src/patch/patch_system.cc @@ -1242,9 +1242,10 @@ else error_exit(PANIC_EXIT, // If no such patch exists, it does an error_exit() (and doesn't return // to the caller). // -// Bugs: -// - This function could be implemented to be very fast, but right now -// it just does a sequential search through all the patches. :( +// FIXME: +// - This function could be implemented to be very fast (using the +// patch numbers in patch_system_info::), but right now it just does +// a sequential search through all the patches, so it's pretty slow :( // const patch& patch_system::plus_or_minus_xyz_patch(bool is_plus, char ctype) const diff --git a/src/patch/patch_system.hh b/src/patch/patch_system.hh index 5f517a3..942f0fc 100644 --- a/src/patch/patch_system.hh +++ b/src/patch/patch_system.hh @@ -117,6 +117,7 @@ public: { return * all_patches_[pn]; } // find a patch by +/- xyz "ctype" + // FIXME: the present implementation of this function is quite slow const patch& plus_or_minus_xyz_patch(bool is_plus, char ctype) const; |