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

Updated version of spelunk.m #3

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

SimonWoods
Copy link

I've made various minor improvements to spelunk.m and am now trying to figure out how to work GitHub :-)

Various minor improvements
@szhorvat
Copy link
Owner

Hi Simon,

Thanks for this! I'm a bit busy now to read through the code (and the diff is not helpful...), so could you tell me if you preserved the modifications that Rob made a while ago? I'd really like to preserve those to have more manageable output cells.

Just take a look at the last few commits to see what he did.

Added comments, improved code structure, added right-click to copy symbol full name to clipboard.
@SimonWoods
Copy link
Author

Hi Szabolcs,

Yes, I preserved Rob's modifications, and also added ShowStringCharacters->True so that string literals appear with quote marks.

I've added a newer version just now - called Spelunking2.m
This one is in much better shape than yesterday's, so you should ignore yesterday's if that's possible (Sorry, I still don't really understand how GitHub works...)

The latest version includes the ability to right-click on a symbol to copy its full name to the clipboard - I put this in because I sometimes find myself wanting to experiment with one of the internal functions that Spelunk discovers and it's a chore to type the context in.

Apart from that, I've improved the code structure and added some comments so it should be easier for others to tinker with it.

I fully understand if you're too busy to deal with this right now - real life sometimes has to take priority over Mathematica!

Simon.

SimonWoods added 7 commits June 27, 2013 23:23
This reverts commit c8fd408.
Now using FrontEnd`UndocumentedTestFEParserPacket to process symbol
definition into boxes. This seems to work better for expressions with
special typesetting.
Using True as the second argument to
FrontEnd`UndocumentedTestFEParserPacket broke the layout. Switched to
False.
Added LoadEverything and SpelunkSearch functions. Refactored code for
easier maintenance.
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.

2 participants