Re: [Midnight Commander] #125: allocate and free memory via mc-wrappers
#125: allocate and free memory via mc-wrappers --+- Reporter: slavazanko | Owner: slavazanko Type: enhancement | Status: testing Priority: major| Milestone: 4.7 Component: mc-core | Version: 4.6.1 Resolution: wontfix |Keywords: memory Blocking: 94 | Blockedby: 157 --+- Changes (by slavazanko): * status: accepted = testing * resolution: = wontfix Comment: I think, need to fix this in any case, but in this ticket now I see proposal of MHL2... not good :) -- Ticket URL: www.midnight-commander.org/ticket/125#comment:10 Midnight Commander www.midnight-commander.org Midnight Development Center ___ Mc-devel mailing list http://mail.gnome.org/mailman/listinfo/mc-devel
Re: [Midnight Commander] #125: allocate and free memory via mc-wrappers
#125: allocate and free memory via mc-wrappers --+- Reporter: slavazanko | Owner: slavazanko Type: enhancement | Status: closed Priority: major| Milestone: 4.7 Component: mc-core | Version: 4.6.1 Resolution: wontfix |Keywords: memory Blocking: 94 | Blockedby: 157 --+- Changes (by winnie): * status: testing = closed -- Ticket URL: www.midnight-commander.org/ticket/125#comment:11 Midnight Commander www.midnight-commander.org Midnight Development Center ___ Mc-devel mailing list http://mail.gnome.org/mailman/listinfo/mc-devel
Re: [Midnight Commander] #125: allocate and free memory via mc-wrappers
#125: allocate and free memory via mc-wrappers --+- Reporter: slavazanko | Owner: slavazanko Type: enhancement | Status: accepted Priority: major| Milestone: 4.7 Component: mc-core | Version: 4.6.1 Resolution: |Keywords: memory Blocking: | Blockedby: --+- Comment(by metux): meanwhile I've moved some steps further: some .h files with macros+inlines in the ./mhl/ subdir (micro helper library). for example mhl/memory.h provides things like mhl_mem_alloc_u(): alloc on heap, unitialized mhl_mem_alloc_z(): alloc on heap, zero'd mhl_mem_free(): free from heap mhl_stack_alloc(): allocate on stack (uninitialized) and mhl/string.h has eg: mhl_str_dup(): safe strdup() (NULL-) mhl_str_ndup(): safe strndup() mhl_str_concat_1(): cat 1 string onto some base str mhl_str_concat_2(): cat 2 strings onto some base str ... -- Ticket URL: www.midnight-commander.org/ticket/125#comment:6 Midnight Commander www.midnight-commander.org Midnight Development Center ___ Mc-devel mailing list http://mail.gnome.org/mailman/listinfo/mc-devel
Re: [Midnight Commander] #125: allocate and free memory via mc-wrappers
#125: allocate and free memory via mc-wrappers --+- Reporter: slavazanko | Owner: slavazanko Type: enhancement | Status: accepted Priority: major| Milestone: 4.7 Component: mc-core | Version: 4.6.1 Resolution: |Keywords: memory Blocking: | Blockedby: --+- Comment(by slavazanko): I think, in future need to move into ./lib dir - more sense place for library... -- Ticket URL: www.midnight-commander.org/ticket/125#comment:7 Midnight Commander www.midnight-commander.org Midnight Development Center ___ Mc-devel mailing list http://mail.gnome.org/mailman/listinfo/mc-devel
Re: [Midnight Commander] #125: allocate and free memory via mc-wrappers
#125: allocate and free memory via mc-wrappers --+- Reporter: slavazanko | Owner: slavazanko Type: enhancement | Status: accepted Priority: major| Milestone: 4.6.2 Component: mc-core | Version: 4.6.1 Resolution: |Keywords: memory Blocking: | Blockedby: --+- Changes (by slavazanko): * owner: = slavazanko * status: new = accepted Comment: Is we need to make these changes in this patch or in later refactoring process? P.S. branch '125-allocate-and-free-memory-via-wrapper' was created. -- Ticket URL: www.midnight-commander.org/ticket/125#comment:3 Midnight Commander www.midnight-commander.org Midnight Development Center ___ Mc-devel mailing list http://mail.gnome.org/mailman/listinfo/mc-devel
Re: [Midnight Commander] #125: allocate and free memory via mc-wrappers
#125: allocate and free memory via mc-wrappers --+- Reporter: slavazanko | Owner: slavazanko Type: enhancement | Status: accepted Priority: major| Milestone: 4.7 Component: mc-core | Version: 4.6.1 Resolution: |Keywords: memory Blocking: | Blockedby: --+- Changes (by winnie): * milestone: 4.6.2 = 4.7 Comment: Move this to a later milestone as this is really a restructurement of mc atm and the 4.6 series should be only a pure bugfix release. This should go only to master not to mc-4.6 branch. -- Ticket URL: www.midnight-commander.org/ticket/125#comment:4 Midnight Commander www.midnight-commander.org Midnight Development Center ___ Mc-devel mailing list http://mail.gnome.org/mailman/listinfo/mc-devel
Re: [Midnight Commander] #125: allocate and free memory via mc-wrappers
#125: allocate and free memory via mc-wrappers --+- Reporter: slavazanko | Owner: slavazanko Type: enhancement | Status: accepted Priority: major| Milestone: 4.7 Component: mc-core | Version: 4.6.1 Resolution: |Keywords: memory Blocking: | Blockedby: --+- Comment(by slavazanko): What should this mcglib actually contain ? It's only for example ;) This should go only to master not to mc-4.6 branch. agree. -- Ticket URL: www.midnight-commander.org/ticket/125#comment:5 Midnight Commander www.midnight-commander.org Midnight Development Center ___ Mc-devel mailing list http://mail.gnome.org/mailman/listinfo/mc-devel
Re: [Midnight Commander] #125: allocate and free memory via mc-wrappers
#125: allocate and free memory via mc-wrappers --+- Reporter: slavazanko | Owner: Type: enhancement | Status: new Priority: major| Milestone: 4.6.2 Component: mc-core | Version: 4.6.1 Resolution: |Keywords: memory Blocking: | Blockedby: --+- Comment(by Enrico Weigelt): * Ticket System tick...@midnight-commander.org schrieb: ACK. But I'd like to go some steps further: * introduce new macros MC_MEM_ALLOC_Z()- alloc on heap, zero'ing MC_MEM_ALLOC_U()- alloc on heap, uninitialized MC_MEM_FREE() - free from stack MC_MEM_REALLOC()- re-allocate (added space uninitialized) MC_STACK_ALLOC()- allocate on stack * replace all other calls to malloc()+friends, mc_alloc()+co, by calling these macros Could also be put into src/util.h (BTW: we dont need any .c file for that) ACK. What should this mcglib actually contain ? cu -- Ticket URL: www.midnight-commander.org/ticket/125#comment: Midnight Commander www.midnight-commander.org Midnight Development Center ___ Mc-devel mailing list http://mail.gnome.org/mailman/listinfo/mc-devel
Re: [Midnight Commander] #125: allocate and free memory via mc-wrappers
#125: allocate and free memory via mc-wrappers --+- Reporter: slavazanko | Owner: Type: enhancement | Status: new Priority: major| Milestone: 4.6.2 Component: mc-core | Version: 4.6.1 Resolution: |Keywords: memory Blocking: | Blockedby: --+- Comment(by slavazanko): related with ticket:94 -- Ticket URL: http://www.midnight-commander.org/ticket/125#comment:1 Midnight Commander www.midnight-commander.org Midnight Development Center ___ Mc-devel mailing list http://mail.gnome.org/mailman/listinfo/mc-devel
