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

Change local labels to a '.' prefix #69

Open
ddevault opened this issue May 14, 2012 · 3 comments
Open

Change local labels to a '.' prefix #69

ddevault opened this issue May 14, 2012 · 3 comments

Comments

@ddevault
Copy link

No description provided.

@enkore
Copy link
Contributor

enkore commented May 29, 2012

Assembler directives start with a dot, labels don't. Doing so would make it nearly impossible to extend the directives in the future.

@ddevault
Copy link
Author

That's not true at all. The regex $[ \t].[A-Za-z0-9_]:[ \t]^ would only match on labels, not directives, and the regex $[ \t].^ would only match on directives, not labels. You just can't use directives like ".directive:". Excuse the funny formatting on this comment, I blame markdown.

@ddevault ddevault reopened this May 29, 2012
@enkore
Copy link
Contributor

enkore commented May 30, 2012

True. Still I think that it would look a bit... confusing, you know.
Let's see, what the other people around here say :-)

That's not true at all. The regex $[ \t].[A-Za-z0-9_]:[ \t]^ would only match on labels, not directives, and the regex $[ \t].^ would only match on directives, not labels. You just can't use directives like ".directive:".

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

No branches or pull requests

2 participants