Jack Ganssle has been doing a series on Coil Cell power at embedded.com :
http://www.embedded.com/electronics-blogs/21/Break-Points
The leakage of the capacitors could be a significant drain to the battery, as Jack discusses.
Also 1N4148 diodes are signal diodes not power diodes. At low currents, such as from a coin cell, this is usually not much of an issue, as long as there enough current to get it to conduct. Running the system power through it could be, an issue, no way to tell without the whole schematic. Taking more than ~75 mA through a 1N914/1N4148 is a bad idea.