basho-labs / riak-c-client

Riak C Client
Apache License 2.0
20 stars 8 forks source link

fix some segfaults while using riak_get() #44

Open folays opened 10 years ago

folays commented 10 years ago
bookshelfdave commented 10 years ago

thanks @folays, we'll try to get to this PR in the next couple of days.

hazen commented 10 years ago

Good catches, @folays. Some of your changes might collide with https://github.com/basho/riak-c-client/issues/40, which is going to clean up the printing interface for all of the messages and structs. Some of the first ones which were written, like GET, are rather cumbersome as you already identified. Also we already have riak_print.c where most of the existing print routines live. We'd want to put new print routines in the same place. As @metadave mentioned, we'll look at it more carefully in the coming days.

bookshelfdave commented 10 years ago

@folays - would you be wiling to submit a new PR to split out the bug fixes vs riak_snprintf_cat() changes? As @javajolt mentioned, we're working on some changes that will be part of #40, which will most likely collide with yours.