opencog-dev team mailing list archive
-
opencog-dev team
-
Mailing list archive
-
Message #00185
Code style changes
[I thought Launchpad would automatically notify everyone, but so far no
mail. So here it is.]
I've applied the style changes to rev 772, pushed it to
lp:~gama/opencog-framework/codestyle and proposed to merge it to
~opencog-dev/opencog-framework/main. It's a *very* boring patch, fixing
a bunch of white space, indentation and bracket issues, but as we're
adopting the practice of reviewing every change, I thought I'd post this
notice anyway.
Linas, I know you're absolutely against this but, please bear with me as
I'm the one who'll have to deal with merges and code reviews very
frequently from now on. Wasting hours every week skipping white space
diffs to find the actual changes is not what I'd call "being
productive".
Besides, I've left the files where only you and Joel are working out of
the patch to avoid sudden changes to your current work flow. If/when
either of you want to join in, just let me know. :-)
If anyone has the time and patience to go through the patch and review
it, I'd appreciate (I strongly suggest using bzr-gtk or another diff
tool as the web version's diff doesn't highlight the white space
changes). Else I'll push it to "main" by the end of the day or tomorrow
morning.
--
Gustavo