Marvin Humphrey wrote on 11/20/09 10:19 PM:

> Task # 2:
> 
>     Again in core Lucy, function names in .c files always use "short names"
>     whenever the "LUCY_USE_SHORT_NAMES" macro is in effect -- e.g. we use
>     "Sort_mergesort" rather than "lucy_Sort_mergesort".  
>     
>     Within Charmonizer, we should be consistent about using short names
>     everywhere the "CHAZ_USE_SHORT_NAMES" macro is in effect, but currently
>     there are lots of places that we don't -- e.g. we use "chaz_Test_plan"
>     rather than "Test_plan" within charmonizer/src/Charmonizer/Test.charm:
> 
>      void
>     -chaz_Test_plan(TestBatch *batch)
>     +Test_plan(TestBatch *batch)
>      {
>          printf("1..%u\n", batch->num_tests);
>      }
> 
>     To see a list of offenders, run the following command from trunk/:
> 
>     find charmonizer/src -print | grep "\.c" | grep -v svn | xargs grep 
> chaz_[A-Z]
> 

I'd be happy to work on this next, as long as it won't step on the other
refactoring going on.


-- 
Peter Karman  .  http://peknet.com/  .  [email protected]

Reply via email to