AU: Class to perform delta updates.

src/platform/update_engine/delta_diff_generator.h

Index: src/platform/update_engine/delta_diff_generator.h
diff --git a/src/platform/update_engine/delta_diff_generator.h b/src/platform/update_engine/delta_diff_generator.h
index 6e48519380d575852aa734d9f0fbd99749298cc3..3cf01a325d1ee4a3d5bf4d0719b240c9799ab745 100644
--- a/src/platform/update_engine/delta_diff_generator.h
+++ b/src/platform/update_engine/delta_diff_generator.h
@@ -111,6 +111,12 @@ class DeltaDiffGenerator {
   DISALLOW_IMPLICIT_CONSTRUCTORS(DeltaDiffGenerator);
 };
 
+namespace {
+const char* const kBsdiffPath = "/usr/bin/bsdiff";

[Daniel Erat, 2010/04/23 17:47:25]

Same objection as in delta_performer.h here -- I don't think that you should be
defining constants at the bottom of a header file (why at the bottom?  why in an
anon namespace?).

[adlr, 2010/04/23 20:50:17]

Sorry, I think I just threw this in while writing code. It seemed like, at least
for integer constant, this might be more efficient since they could be inlined.
the anon namespace was the c++ conversion of 'static', which is how i would have
done it in c.

anyway, changed to non-anon namespace and to just declarations with the
definitions in the corresponding .cc file.

+const char* const kBspatchPath = "/usr/bin/bspatch";
+const char* const kDeltaMagic = "CrAU";
+}
+
 };  // namespace chromeos_update_engine
 
 #endif  // CHROMEOS_PLATFORM_UPDATE_ENGINE_DELTA_DIFF_GENERATOR_H__