Robinson, Paul
2012-Nov-27 23:03 UTC
[LLVMdev] [cfe-dev] RFC: A Great Renaming of Things (or: Let's Repaint ALL the Bikesheds!)
> Are there other names that are poor choices and are lingering in our > codebases? I'm willing to sign up to do more renames while I'm at > this, so this is a chance to get someone else to do the heavy lifting. > =]Class names too, not just file/directory names? I've been trolling through the integrated assembler lately, and had a real "can't tell the players without a program" experience with some class names that really imply the wrong hierarchical relationships. // Doesn't start off too badly. MCObjectWriter ELFObjectWriter MachObjectWriter // But no [Win]COFFObjectWriter (don't expect you to change that!) // Then we get to the target-specific bits... MCELFObjectTargetWriter <arch>ELFObjectWriter // <arch> = X86, MBlaze, ARM, Mips, PPC // Note the subclasses do not have "Target" in the name. // Which makes them look like subclasses of ELFObjectWriter. // And now a special special case. MCWinCOFFObjectTargetWriter // in MCWinCOFFObjectWriter.h X86WinCOFFObjectWriter // Subclass name makes it look like it ought to be a subclass of the // (nonexistent) [Win]COFFObjectWriter. Thanks, --paulr