Re: [OpenWeBWorK-devel] identation patch

[Sam H. <sh...@ma...>]

On Oct 20, 2004, at 5:48 PM, John Jones wrote:

> Sam Hathaway wrote:
>
>> On Oct 20, 2004, at 4:03 PM, John Jones wrote:
>>
>>> I scanning through the recent cvs change to SetMaker.pm, it looks 
>>> like there are places where whitespace was changed inside quoted 
>>> strings (which would be displayed in output).  Was this intentional? 
>>>  There didn't seem to be an conventions in the twiki reference about 
>>> this.
>>
>>
>> No, just me being careless. (On the other hand, whitespace in HTML 
>> output doesn't matter much, since browsers collapse it.) Can you 
>> point to specific places so I can correct them?
>> -sam
>
> The ones which first caught my attention are at the top:
>
> -use constant MY_PROBLEMS => '  My Problems  ';
> -use constant MAIN_PROBLEMS => '  Main Problems  ';
> +use constant MY_PROBLEMS => '	 My Problems	';
> +use constant MAIN_PROBLEMS => '	 Main Problems	';
>
>
> On the plus side, these are constants so that they will be used 
> consistently in the file.  I may adjust others if I happen on them.

Sorry about that. Fixing now...
-sam