Looks good for me. Just, I don't see why we need to change the arguments order there:
@@ -93,7 +93,7 @@ gog_contour_plot_build_matrix (GogXYZPlot const *plot, gboolean *cardinality_cha for (i = 0; i < plot->rows; i++) for (j = 0; j < plot->columns; j++) { val = gog_axis_map_to_view (map, - go_data_get_matrix_value (mat, j, i)); + go_data_get_matrix_value (mat, i, j)); if (fabs (val) == DBL_MAX) val = go_nan; else { Jean Le mardi 21 avril 2009 à 17:16 +0200, Emmanuel Pacaud a écrit : > Hi, > > Here's a second try for the multi-dimensional GOData patch. > > This time, the API compatibility is preserved. And the internal of > GOData is able to handle any number of data dimensions. > > Please tell me what you think of this patch set. > > Emmanuel. > > _______________________________________________ > gnumeric-list mailing list > gnumeric-list@gnome.org > http://mail.gnome.org/mailman/listinfo/gnumeric-list _______________________________________________ gnumeric-list mailing list gnumeric-list@gnome.org http://mail.gnome.org/mailman/listinfo/gnumeric-list