> On Fri, 2008-02-08 at 16:10 +1300, Amos Jeffries wrote:
>
> The "same()" operations used in those tests may be different from a
> "same_MD5s()" operation.
>
> Try inserting an empty line in a middle of a function and recompile. You
> will get a different MD5. Stripping the executable does not help in my
> tests......................

Yes this is true. I tried exactly  the same tests  to evaluate  the
formatter, but they did not work.  I believed that striping the code will
solve the problem but nothing...
In general I was not able to find a safe way to test it. I am just looking
the diffs searching for  bad formated code ...
(Also I must note here that the bazaar repository was very-very  helpful
here, branches/revert/diff etc on local repositories are really fast, but
OK it is an other story :-) )

>................... Stripping empty lines fron sources does not help if
the formatter
> moves brackets and such, changing the number of lines. Does our
> formatter do that?
>
Yes it does. For example the code:
  void some_function(){
  }
convert to:
  void some_function()
 {
  }




Reply via email to