On 19 December 2016 at 20:23, Gustavo Sverzut Barbieri <[email protected]> wrote:
> On Mon, Dec 19, 2016 at 1:07 AM, Jean-Philippe ANDRÉ <[email protected]> > wrote: > > jpeg pushed a commit to branch master. > > > > http://git.enlightenment.org/core/efl.git/commit/?id= > 72f98a06499cbabc7f56290afa89f7b8b37f52b9 > > > > commit 72f98a06499cbabc7f56290afa89f7b8b37f52b9 > > Author: Jean-Philippe Andre <[email protected]> > > Date: Mon Dec 19 12:03:46 2016 +0900 > > > > evas/ply: Fix some memleaks > > > > I have a strong feeling that this parser is not very robust... > > > > Closes T5001 > > --- > > .../evas/model_loaders/ply/evas_model_load_ply.c | 34 > ++++++++++++++++------ > > 1 file changed, 25 insertions(+), 9 deletions(-) > > > > diff --git a/src/modules/evas/model_loaders/ply/evas_model_load_ply.c > b/src/modules/evas/model_loaders/ply/evas_model_load_ply.c > > index eb4228b..9f49039 100644 > > --- a/src/modules/evas/model_loaders/ply/evas_model_load_ply.c > > +++ b/src/modules/evas/model_loaders/ply/evas_model_load_ply.c > > @@ -53,18 +53,20 @@ _read_ply_header(char *map, > Evas_Model_Load_Save_Header *header) > > > > if (helping_pointer == NULL) > > { > > - ERR("File have not kayword vertex. It is necessary."); > > + ERR("PLY file doesn't contain the required keyword 'vertex'"); > > plz put the file path or id... otherwise it's a PITA to know which > file the log is talking about. > Good point, but the name isn't known in this function. All I did was to clean a patch sent on Phab. -- Jean-Philippe André ------------------------------------------------------------------------------ Developer Access Program for Intel Xeon Phi Processors Access to Intel Xeon Phi processor-based developer platforms. With one year of Intel Parallel Studio XE. Training and support from Colfax. Order your platform today.http://sdm.link/intel _______________________________________________ enlightenment-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
