Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Pointers to Axis and Error_bars #142

Closed
RaFaeL-NN opened this issue Jan 15, 2018 · 3 comments
Closed

Pointers to Axis and Error_bars #142

RaFaeL-NN opened this issue Jan 15, 2018 · 3 comments
Assignees
@jmcnamara
Labels
feature request ready to close

Comments

@RaFaeL-NN
Copy link

RaFaeL-NN commented Jan 15, 2018 •
edited by jmcnamara

To use the libxlsxwriter in languages other than C as external dll (where C headers can not be used), it would be better to add methods that return pointers to axis and error_bars, something like

lxw_chart_axis * chart_get_axis (lxw_chart * chart, uint8_t type)
lxw_series_error_bars * chart_series_get_error_bars(lxw_chart_series * series,  uint8_t type)

(type means X or Y)

So, it will simply use these pointers in chart_series_set_error_bars*and chart_axis_set* methods.
@jmcnamara jmcnamara self-assigned this Jan 15, 2018
jmcnamara added a commit that referenced this issue Feb 11, 2018
@jmcnamara
Add functions to access axis and error bar pointers.
d863c8e 
Add functions to return pointers to chart axis and error bar
pointers as syntactic sugar for wrapper languages that can't
access the pointers directly.

Issue #142
@jmcnamara
Copy link
Owner

I've push a fix for this to the master branch. See commit d863c8e.

@RaFaeL-NN
Copy link
Author

Works good, thanks

@jmcnamara
Copy link
Owner

Added in version 0.7.6. Closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jmcnamara jmcnamara

Labels
feature request ready to close
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jmcnamara @RaFaeL-NN