On 2013-09-20, at 5:55 PM, Pascal Quantin <pascal.quan...@gmail.com> wrote:

> Hi all,
> 
> the easy part of the conversion from emem to wmem memory should be almost 
> complete now: dissectors and plugins use the new memory manager (with the 
> exception of uat / initialization routines).
> 
> Next, I was thinking about converting our helper functions found in epan 
> module and the various ep_ and se_ functions.
> My idea was to update (and rename!) those functions with the following scheme:
> tvb_get_ephemeral_string -> tvb_get_wmem_packet_string
> tvb_get_seasonal_string -> tvb_get_wmem_file_string
> and adding defines to maintain (temporarily?) backward compatibility:
> #define tvb_get_ephemeral_string tvb_get_wmem_packet_string
> #define tvb_get_seasonal_string tvb_get_wmem_file_string
> 
> Of course we will discover some misuse of ep/se memory (like for uat) that 
> will require more thinking, but it would be one step forward.
> 

I was hoping (at least for functions with both an ep and se version) to make 
the pool a parameter, so for example

tvb_get_string(pool, ...)

This can also replace the glib versions of such functions automatically by 
passing NULL as the pool (wmem provides this "for free").

The macros are a good idea.

> Thoughts?
> 
> Regards,
> Pascal.
> ___________________________________________________________________________
> Sent via:    Wireshark-dev mailing list <wireshark-dev@wireshark.org>
> Archives:    http://www.wireshark.org/lists/wireshark-dev
> Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
>             mailto:wireshark-dev-requ...@wireshark.org?subject=unsubscribe
___________________________________________________________________________
Sent via:    Wireshark-dev mailing list <wireshark-dev@wireshark.org>
Archives:    http://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
             mailto:wireshark-dev-requ...@wireshark.org?subject=unsubscribe

Reply via email to