<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><div>Thanks for explaining, John. This should work fine regardless of indentation size as long as the tab stops in your editor are 8 wide (which may or may not be global truth).</div><div><br></div><div>Kelly: Looks good!</div><div><br></div><div>/Staffan</div><br><div><div>On 3 mar 2012, at 14:55, John Rose wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><div><div>Lest Kelly have all the fun, I'll jump in.</div><div><br></div><div>On Mar 3, 2012, at 3:36 AM, David Holmes wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><span class="Apple-style-span" style="border-collapse: separate; font-family: Helvetica; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-align: -webkit-auto; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-border-horizontal-spacing: 0px; -webkit-border-vertical-spacing: 0px; -webkit-text-decorations-in-effect: none; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; font-size: medium; ">So the question is: what does the script think a TAB represents?<br></span></blockquote></div><br><div>The same thing that /usr/bin/expand does, as noted earlier. Or read the perl code.</div><div><br></div><div>When was the last time anybody on the hotspot team used '\t' (the source file octet, not the keyboard key) to mean anything else than /usr/bin/expand? This stuff about 2 and 4 width indents is irrelevant, except to people who accidentally use anti-social IDE settings, and they get socialized quickly.</div><div><br></div><div><div>On Mar 3, 2012, at 8:39 AM, Jonathan Gibbons wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div>Kelly,<br><br>Is there a reason you don't use "expand"?<br><br>For a while now I've been using the following simple script to fix the whitespace in my files before I commit a change set.<br></div></blockquote></div><br><div>The perl script does what your script does, probably to 95% compatibility. (File list generation logic and last-line behavior may perhaps vary on corner cases.) Your script uses perl plus other shell commands, while the standard script uses perl only; your script uses expand instead of an obscure couple of perl lines which do the same thing. It seems a matter of taste not greatly worth discussing. There are 10 different ways to code this operation; 9 of those ways will not get used, but (IMO) it's not very interesting to ask why not.</div><div><br></div><div>— John</div></div></blockquote></div><br></body></html>