Code review comment for lp:~albertog/siesta/gridxc-integration

Revision history for this message
Nick Papior (nickpapior) wrote :

I think it looks good.

However, it seems that there is going to be needed "tons" of handlers.f in all utilities.
Wouldn't it be cleaner to add the handlers.f in the Src directory to not have the code dublicated.
We may then call it handlers_utils.f (or just handlers) to make it accessible for all the utilities?

« Back to merge proposal