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

Conversion to bases <=1 or >10 fails #68

Closed
refi64 opened this issue May 1, 2016 · 8 comments
Closed

Conversion to bases <=1 or >10 fails #68

refi64 opened this issue May 1, 2016 · 8 comments
Assignees

Comments

@refi64
Copy link
Collaborator

refi64 commented May 1, 2016

Stuff like 21b or 11b will loop indefinitely.

@refi64 refi64 added the bug label May 1, 2016
@refi64
Copy link
Collaborator Author

refi64 commented May 1, 2016

Huh, and ACb results in...:!? This needs a bit of rework. :O

@refi64 refi64 changed the title Infinite loop when converting to base 1 (unary) Conversion to bases <=1 or >10 fails May 1, 2016
@refi64
Copy link
Collaborator Author

refi64 commented May 1, 2016

@phase What if we changed b to return an array of numbers instead of a string?

@phase
Copy link
Owner

phase commented May 1, 2016

If we have a array of number to string operator then sure.

As a side note, I'll start working on O again once I get my laptop working again. Crashing during an update does wonders. Shouldn't take too much longer.

@refi64
Copy link
Collaborator Author

refi64 commented May 1, 2016

@phase

If we have a array of number to string operator then sure.

Well, you could do something like [[1234]{n""+}d]p...

As a side note, I'll start working on O again once I get my laptop working again. Crashing during an update does wonders. Shouldn't take too much longer.

Ouch. Was that a Windows crash? I actually haven't been able to get the Windows side of my laptop to boot in forever, so I can sort of related to that. :/

@phase
Copy link
Owner

phase commented May 1, 2016

An operator would be better, I'm surprised no one noticed O doesn't have one.

I switched to Kubuntu a while ago, and I was updating to Ubuntu 16 when KDE decided to do its hourly crash. I'm backing my root folder up to a hard drive and I'm just going to reinstall it.

@refi64
Copy link
Collaborator Author

refi64 commented May 1, 2016

@phase Yeah, I need to upgrade mine. Still on Ubuntu 14 LTS. I'm surprised you stuck with Kubuntu if it crashed hourly...

@refi64
Copy link
Collaborator Author

refi64 commented May 1, 2016

@phase I got an idea. What if range (,) gets overloading so that, if the TOS is a string, then it will join the next value on the stack with the string as the separator? So that ['1'2'3'4'5]S, would output 12345. It would convert any non-string values to strings, so [12345]S, would be the same thing.

@phase
Copy link
Owner

phase commented May 1, 2016

Ooo that's nice.

Plasma 5 is just a little finicky if you mess with it a lot. Luckily it has a built in error catcher, so you can instantly restart it whenever you decide to mess with the settings.

refi64 added a commit that referenced this issue May 1, 2016
@refi64 refi64 closed this as completed in c66d7a9 May 21, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants