On Fri, Feb 12, 2016 at 7:27 AM Daniel Kinzler <dan...@brightbyte.de> wrote:
> CON: don't do mass migration to new syntax, only start using new styles and > features when touching the respective bit of code anyway. The argument is > here > that touching many lines of code, even if it's just for whitespace changes, > causes merge conflicts when doing backports and when rebasing patches. > E.g. if > we touch half the files in the codebase to change to the new array syntax, > who > is going to manually rebase the couple of hundred patches we have open? > > > As can be seen on the proposed patch I linked, several of the long term > developers oppose mass changes like this. A quick round of feedback in the > architecture committee draws a similar picture. However, perhaps there are > compelling arguments for doing the mass migration that we haven't heard > yet. So > please give a quick PRO or CON, optionally with some rationale. > > My personal vote is CON. No rebase hell please! Changing to the syntax > doesn't > buy us anything. > > CON, for all the reasons you mentioned. Also: style only changes are pain when you're trying to annotate/blame a particular line of code. ESPECIALLY for something so silly as array formatting which gains us *absolutely nothing* -Chad _______________________________________________ Wikitech-l mailing list Wikitech-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikitech-l