Changes between Initial Version and Version 1 of PreDecemberReleaseHackathon

Show
Ignore:
Timestamp:
12/12/09 02:35:19 (12 years ago)
Author:
jkeenan
Comment:

Resources for Dec 12 2009 Development Hackathon

Legend:

Unmodified
Added
Removed
Modified
  • PreDecemberReleaseHackathon

    v1 v1  
     1= December 12 2009 Pre-Release Hackathon = 
     2 
     3== Topics == 
     4 
     5 * 12 December 2009: Miscellaneous Hackathon - We'll focus on: 
     6   * Merge CallSignature and Context. This work is being done in the context_unify3 branch. 
     7   * Remove uses of ->strstart outside of src/string.  Either use Parrot_str_to_cstring/Parrot_str_free_cstring bufstart.  See r40983 for examples of both and ask on #parrot if you're not sure. 
     8   * When in doubt, look at Parrot's [http://tapir2.ro.vutbr.cz/cover/cover-results/42986/c_cover/ test coverage] and write tests for code we don't currently cover. 
     9 
     10== Resources == 
     11 
     12=== `->strstart` cleanup === 
     13 
     14Here are 73 instances of `->strstart` in 19 files outside of ''src/string/''.[[BR]] 
     15(Note:  Wiki formatting problems mean that the code listed for ''./src/pmc/filehandle.pmc'' line 370 is not correct.) 
     16 
     17Check off completed instances by editing wiki to put an asterisk after first 2 vertical bars. 
     18 
     19||||./src/ops/string.ops||392||$1 = PTR2UINTVAL($2->strstart); 
     20||||./src/gc/api.c||527||string->strstart        = NULL; 
     21||||./src/gc/api.c||734||sets also C<< str->strstart >> to the new buffer location, C<< str->bufused >> 
     22||||./src/gc/api.c||764||Buffer_bufstart(str) = str->strstart = mem; 
     23||||./src/gc/api.c||830||oldmem             = str->strstart; 
     24||||./src/gc/api.c||832||str->strstart      = mem; 
     25||||./src/gc/alloc_resources.c||439||offset = (ptrdiff_t)((STRING *)b)->strstart - 
     26||||./src/gc/alloc_resources.c||464||((STRING *)b)->strstart = (char *)Buffer_bufstart(b) + 
     27||||./src/gc/alloc_resources.c||500||((STRING *)b)->strstart = (char *)Buffer_bufstart(b) + 
     28||||./src/gc/alloc_resources.c||910||PARROT_ASSERT(((STRING *) pobj)->strstart >= 
     29||||./src/gc/alloc_resources.c||912||PARROT_ASSERT(((STRING *) pobj)->strstart + 
     30||||./src/gc/alloc_resources.c||927||PARROT_ASSERT(((STRING *)pobj)->strstart >= 
     31||||./src/gc/alloc_resources.c||929||PARROT_ASSERT(((STRING *)pobj)->strstart + 
     32||||./src/gc/mark_sweep.c||312||(char*) ((PMC*)b)->vtable->whoami->strstart); 
     33||||./src/packfile/pf_items.c||1331||s->strstart, OFFS(pf, *cursor))); 
     34||||./src/packfile/pf_items.c||1389||if (s->strstart) { 
     35||||./src/packfile/pf_items.c||1390||mem_sys_memcopy(charcursor, s->strstart, s->bufused); 
     36||||./src/pmc/filehandle.pmc||370||char * const r = readline(got_prompt ? prompt->strstart NULL); 
     37||||./src/pmc/filehandle.pmc||381||fprintf(stderr, "%s", prompt->strstart); 
     38||||./src/pmc/filehandle.pmc||394||&&    (((char *)string_result->strstart)[len - 1] == '\n' 
     39||||./src/pmc/filehandle.pmc||395||   ((char *)string_result->strstart)[len - 1] == '\r')) { 
     40||||./src/pmc/eval.pmc||284||if ((size_t)(res->strstart) & 0xf) { 
     41||||./src/pmc/eval.pmc||285||char *adr     = res->strstart; 
     42||||./src/pmc/eval.pmc||287||res->strstart = adr; 
     43||||./src/pmc/eval.pmc||296||PackFile_pack(INTERP, pf, (opcode_t *)res->strstart); 
     44||||./src/pmc/eval.pmc||354||if (!PackFile_unpack(INTERP, pf, (opcode_t *)packed->strstart, 
     45||||./src/io/win32.c||493||void * const buffer = s->strstart; 
     46||||./src/io/unix.c||556||const char * const buffer = s->strstart; 
     47||||./src/io/utf8.c||64||const utf8_t *u8ptr = (utf8_t *)((char *)s->strstart + 
     48||||./src/io/portable.c||306||const size_t   bytes = fread(s->strstart, 1, 1, fptr); 
     49||||./src/io/portable.c||311||ungetc(*(char *)s->strstart, fptr); 
     50||||./src/io/portable.c||409||void * const buffer = s->strstart; 
     51||||./src/io/buffer.c||303||if (!s->strstart) 
     52||||./src/io/buffer.c||306||out_buf = (unsigned char *)s->strstart; 
     53||||./src/io/buffer.c||442||memmove(s->strstart, buffer_next, len); 
     54||||./src/io/buffer.c||510||if (s->strstart) { 
     55||||./src/io/buffer.c||517||out_buf = (unsigned char*)s->strstart + s->strlen; 
     56||||./src/io/buffer.c||529||if (s->strstart) { 
     57||||./src/io/buffer.c||536||out_buf = (unsigned char*)s->strstart + s->strlen; 
     58||||./src/io/buffer.c||568||void          * const buffer       = s->strstart; 
     59||||./src/io/socket_win32.c||251||if ((error = send((int)io->os_handle, (char *)s->strstart + byteswrote, 
     60||||./src/io/socket_unix.c||304||if ((error = send(io->os_handle, (char *)s->strstart + byteswrote, 
     61||||./src/utils.c||585||const char * const str_start  = base->strstart; 
     62||||./src/utils.c||587||const char * const search_str = search->strstart; 
     63||||./src/utils.c||635||const char * const search_start = search->strstart; 
     64||||./src/utils.c||644||const char * const base_start = (char *)base->strstart + current_offset; 
     65||||./src/oo.c||958||bits = (((UINTVAL) method_name->strstart) >> 2) & TBL_SIZE_MASK; 
     66||||./src/oo.c||979||while (e && e->strstart != method_name->strstart) { 
     67||||./src/oo.c||991||e->strstart = method_name->strstart; 
     68||||./src/dynext.c||335||if (!STRING_IS_EMPTY(lib) && memcmp(lib->strstart, "lib", 3) == 0) { 
     69||||./src/dynext.c||336||*handle = Parrot_dlopen((char *)lib->strstart + 3); 
     70||||./src/dynext.c||346||if (!STRING_IS_EMPTY(lib) && memcmp(lib->strstart, "lib", 3) == 0) { 
     71||||./src/pmc_freeze.c||348||ignored = PF_store_number((opcode_t *)((ptrcast_t)s->strstart + used), &v); 
     72||||./src/pmc_freeze.c||377||ignored = PF_store_string((opcode_t *)((ptrcast_t)s->strstart + used), v); 
     73||||./src/pmc_freeze.c||399||const char * const   start  = (char *)io->image->strstart; 
     74||||./src/pmc_freeze.c||400||char               **opcode = &io->image->strstart; 
     75||||./src/pmc_freeze.c||404||io->image->bufused -= ((char *)io->image->strstart - start); 
     76||||./src/pmc_freeze.c||405||io->image->strlen  -= ((char *)io->image->strstart - start); 
     77||||./src/pmc_freeze.c||428||const char * const   start  = (const char *)io->image->strstart; 
     78||||./src/pmc_freeze.c||429||char               **opcode = &io->image->strstart; 
     79||||./src/pmc_freeze.c||433||io->image->bufused -= ((char *)io->image->strstart - start); 
     80||||./src/pmc_freeze.c||434||io->image->strlen  -= ((char *)io->image->strstart - start); 
     81||||./src/pmc_freeze.c||459||char * const   start  = (char*)io->image->strstart; 
     82||||./src/pmc_freeze.c||460||char *         opcode = io->image->strstart; 
     83||||./src/pmc_freeze.c||464||io->image->strstart = opcode; 
     84||||./src/pmc_freeze.c||535||mem_sys_memcopy(s->strstart, pf->header, PACKFILE_HEADER_BYTES); 
     85||||./src/pmc_freeze.c||554||mem_sys_memcopy(pf->header, s->strstart, PACKFILE_HEADER_BYTES); 
     86||||./src/pmc_freeze.c||560||LVALUE_CAST(char *, s->strstart) += header_length; 
     87||||./src/pmc_freeze.c||1140||LVALUE_CAST(char *, image->strstart) -= bufused; 
     88||||./src/pmc_freeze.c||1144||PARROT_ASSERT(image->strstart >= (char *)Buffer_bufstart(image)); 
     89||||./t/op/string_cs.t||735||# 106         dest_len = u_strToUpper(src->strstart, dest_len, 
     90||||./t/op/string_cs.t||740||# (gdb) x /8h src->strstart 
     91||||./t/op/string_cs.t||746||# (gdb) x /8h src->strstart