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.

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

[valarray.members] add valarray index entry for named member functions #791

Closed

Conversation

AlisdairM
Copy link
Contributor

The index currently lists the named member-functions of valarray as
if they were free functions - this adds the corresponding entry under
the valarray listing in the index, alongside the existing entries for
the many operator functions.

@AlisdairM
Copy link
Contributor Author

sorry - looks like I have a pull request based on a reverted version of 'master'?

@tkoeppe
Copy link
Contributor

tkoeppe commented Jul 4, 2016

@AlisdairM master got force-pushed, so you too should fetch, rebase and force push.

The index currently lists the named member-functions of valarray as
if they were free functions - this adds the corresponding entry under
the valarray listing in the index, alongside the existing entries for
the many operator functions.
@AlisdairM
Copy link
Contributor Author

rebased off the current master and force-pushed

@tkoeppe
Copy link
Contributor

tkoeppe commented Jul 9, 2016

(Don't tell anyone, but we had one or two force-pushes to master in the last week. Sorry about the inconvenience.)

@@ -7429,6 +7429,7 @@
\end{itemdescr}

\indexlibrary{\idxcode{length}!\idxcode{valarray}}%
\indexlibrary{\idxcode{valarray}!\idxcode{length}}%
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why does this say "length" when the function is called "size"?

@tkoeppe
Copy link
Contributor

tkoeppe commented Jul 9, 2016

I stole this PR; it is now #808.

@tkoeppe tkoeppe closed this Jul 9, 2016
@AlisdairM AlisdairM deleted the index_valarray_functions branch July 9, 2016 12:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants