[pulseaudio-discuss] [PATCH 1/2] Add a target to the PA log feature

Colin Guthrie gmane at colin.guthr.ie
Thu Feb 24 16:28:18 PST 2011


'Twas brillig, and Paul Menzel at 24/02/11 22:14 did gyre and gimble:
> Could you add that indentation change in a separate patch? It makes it
> easier to review in my opinion. It is not necessary though.

I think we'll try and avoid the majority of the indentation changes if
possible. I appreciate why but it would be better to just move to a new
line for the case where the indentation is needed rather than change
everything.

People still like 80 char limits too.

Col

-- 

Colin Guthrie
gmane(at)colin.guthr.ie
http://colin.guthr.ie/

Day Job:
  Tribalogic Limited [http://www.tribalogic.net/]
Open Source:
  Mageia Contributor [http://www.mageia.org/]
  PulseAudio Hacker [http://www.pulseaudio.org/]
  Trac Hacker [http://trac.edgewall.org/]




More information about the pulseaudio-discuss mailing list