[pisa-src] PISA code prefix

  • From: Thomas Jansen <mithi@xxxxxxxxx>
  • To: pisa-src@xxxxxxxxxxxxx
  • Date: Fri, 2 Oct 2009 15:33:04 +0200

docs/HACKING defines ps_ as the prefix every PISA-specific part of the code
should use. Right now most (but not all) of the code uses pisa_ as a common
prefix.

There is a simple solution for updating the code to the standards of
docs/HACKING: find . -name '*.[h|c]' -exec sed -i "s/pisa_/ps_/g" '{}' \;
It compiles and works at first glance, but produces a huge diff. Since I know
Diego and his dislike for these things, I'd like to gather some feedback and
opinions from all of you before deciding on a course of action. Basically the
options are:
1. Leave it alone.
2. Convert pisa_ to ps_ with the command mentioned above.
3. Convert pisa_ to ps_ function by function.

Downsides are in my opinion:
1. Inconsistency
2. One huge commit with a probably unreadable diff.
3. Huge number of commits, each addressing only one name and therefore
changing one line multiple times in a row. Also needs more effort.

I'm in favor of option 2 unless someone comes up with another (feasible) idea.
-- 
Thomas Jansen, "Mithi" --- mithi@xxxxxxxxx
GPG 9D5C682B, feel free to sign or encrypt your mail

Other related posts: