Skip to content

Add '<' meta command to read in statements from external file #1783

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

Closed

Conversation

pcbeard
Copy link

@pcbeard pcbeard commented Sep 11, 2020

Perform all error handling in ReadCode()

Perform all error handling in ReadCode()
This loads A.swift into the REPL and executes code defined therein
@pcbeard pcbeard requested a review from jimingham September 11, 2020 08:38
Copy link

@jimingham jimingham left a comment

Choose a reason for hiding this comment

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

This looks great.

Can I ask you to do one more thing? In source/Expression/REPL.cpp there is a bit of text that the REPL emits when you run :help that gives an intro to REPL specific uses. It's in the function IOHandlerGetHelpPrologue. Can you add a brief sentence describing the use of the initial < meta-character? Otherwise I don't know how anyone would discover this new feature...

Copy link

@jimingham jimingham left a comment

Choose a reason for hiding this comment

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

LGTM

@pcbeard pcbeard requested a review from fredriss September 11, 2020 23:29
@pcbeard pcbeard removed the request for review from fredriss September 11, 2020 23:29
@pcbeard
Copy link
Author

pcbeard commented Sep 14, 2020

This will need to be split up. Most of it will need to go straight to llvm:master, the tests will remain in apple:swift/master.

@pcbeard
Copy link
Author

pcbeard commented Sep 25, 2020

This PR is subsumed by:

#1856
#1857
#1858

@pcbeard pcbeard closed this Sep 25, 2020
@pcbeard pcbeard deleted the pcbeard/PR-RedirectReplInput branch September 25, 2020 20:23
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.

3 participants