<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.0 Transitional//EN">
<HTML><HEAD>
<META http-equiv=Content-Type content="text/html; charset=us-ascii">
<META content="MSHTML 6.00.6001.18226" name=GENERATOR></HEAD>
<BODY>
<DIV><SPAN class=671120223-28052009><FONT face="Lucida Sans"><FONT 
color=#000080><FONT size=2><SPAN class=879491523-28052009>Mondrian &amp; olap4j 
developers,</SPAN></FONT></FONT></FONT></SPAN></DIV>
<DIV><SPAN class=671120223-28052009><FONT face="Lucida Sans"><FONT 
color=#000080><FONT size=2><SPAN 
class=879491523-28052009></SPAN></FONT></FONT></FONT></SPAN>&nbsp;</DIV>
<DIV><SPAN class=671120223-28052009><FONT><FONT><SPAN 
class=879491523-28052009></SPAN><FONT face="Lucida Sans" color=#000080 size=2>A 
few days ago I changed the&nbsp;<SPAN class=879491523-28052009>c</SPAN>heckFile 
script to enforce line length. I also fixed&nbsp;<SPAN 
class=879491523-28052009>all olap4j file, and </SPAN>a few&nbsp;<SPAN 
class=879491523-28052009>mondrian </SPAN>files, to comply with the 80 character 
limit. This has always been the standard; we've just not enforced it 
before.</FONT></FONT></FONT></SPAN></DIV>
<DIV><SPAN class=671120223-28052009><FONT face="Lucida Sans" color=#000080 
size=2></FONT></SPAN>&nbsp;</DIV>
<DIV><SPAN class=671120223-28052009><FONT face="Lucida Sans" color=#000080 
size=2>Because I wasn't able to fix all&nbsp;<SPAN 
class=879491523-28052009>mondrian </SPAN>files,&nbsp;<SPAN 
class=879491523-28052009>c</SPAN>heckFile allows line lengths up to 9<SPAN 
class=879491523-28052009>0</SPAN> for existing files, but in strict mode - which 
is invoked if you use the '--opened' or '--strict' flag - lines must be no 
longer than 80 chars.</FONT></SPAN></DIV>
<DIV><SPAN class=671120223-28052009><FONT face="Lucida Sans" color=#000080 
size=2></FONT></SPAN>&nbsp;</DIV>
<DIV><SPAN class=671120223-28052009><FONT face="Lucida Sans" color=#000080 
size=2>What do you need to do?&nbsp;<SPAN class=879491523-28052009>Before 
checking in a file, r</SPAN>un&nbsp;<SPAN class=879491523-28052009>'</SPAN><SPAN 
class=879491523-28052009>c</SPAN>heckFile --opened' and fix all violations. You 
may need to fix a few violations which were not your fault.&nbsp;<SPAN 
class=879491523-28052009>Sorry about that. </SPAN>But you will be getting us 
nearer to&nbsp;asymptotic compliance with the coding 
guidelines.</FONT></SPAN></DIV>
<DIV><SPAN class=671120223-28052009><FONT face="Lucida Sans" color=#000080 
size=2></FONT></SPAN>&nbsp;</DIV>
<DIV><SPAN class=671120223-28052009><SPAN class=879491523-28052009><FONT 
face="Lucida Sans" color=#000080 size=2>If I send review comments, please fix 
them in a timely manner. If you do not, I will back out your change or suspend 
your committer privileges.</FONT></SPAN></SPAN></DIV>
<DIV><SPAN class=671120223-28052009><SPAN class=879491523-28052009><FONT 
face="Lucida Sans" color=#000080 size=2></FONT></SPAN></SPAN>&nbsp;</DIV>
<DIV><SPAN class=671120223-28052009><SPAN class=879491523-28052009><FONT 
face="Lucida Sans" color=#000080 size=2>Sorry to be authoritarian. It's 
important that we have consistency in code formatting. We tried making 
compliance voluntary, and it didn't work.</FONT></SPAN></SPAN></DIV>
<DIV><SPAN class=671120223-28052009><SPAN class=879491523-28052009><FONT 
face="Lucida Sans" color=#000080 size=2></FONT></SPAN></SPAN>&nbsp;</DIV>
<DIV><SPAN class=671120223-28052009><SPAN class=879491523-28052009>
<DIV><SPAN class=671120223-28052009><SPAN class=879491523-28052009><FONT 
face="Lucida Sans" color=#000080 size=2>For details on getting checkFile, and 
developer's guidelines in general, please see <A 
href="http://mondrian.pentaho.org/documentation/developers_guide.php#The_checkFile_utility"><FONT 
face="Times New Roman" 
size=3>http://mondrian.pentaho.org/documentation/developers_guide.php#The_checkFile_utility</FONT></A>.</FONT></SPAN></SPAN></DIV></SPAN></SPAN></DIV>
<DIV><SPAN class=671120223-28052009><FONT face="Lucida Sans" color=#000080 
size=2></FONT></SPAN>&nbsp;</DIV>
<DIV><SPAN class=671120223-28052009><FONT face="Lucida Sans" color=#000080 
size=2>Julian</FONT></SPAN></DIV></BODY></HTML>