search

Tomivix / hpa

First release (v0.9.9) is available here: https://github.com/Tomivix/hpa/raw/master/rel/HPA_Studio_0.9.9_%5B576235A3%5D.jar
Other
5 stars 2 forks source link

Adding option to save and read from file #30

Closed Tomivix closed 7 years ago

Tomivix commented 7 years ago

This is a pretty dirty version of file saving/reading. Doesn't have any real means of validating the selected file but it should do the trick for now. I will improve it later today but that's all I can offer before KNI meeting.

Tomivix commented 7 years ago

Waiting for your opinions @Aicedosh and @mrwasp. I still should be able to make some small changes before the meeting if you see any problems. Also does anyone have a notebook which isn't as crappy as mine and has Java RE installed? Relying on organizers for that part usually isn't the best idea...

mrwasp commented 7 years ago

One bug found: in save you have "\<\<\<SEPARATOR>>" but in load "\<\<\<SEPARATOR>>>", which causes that we aren't able to load previously saved file. All the rest works enough good by now.

Tomivix commented 7 years ago

Yeah, right... I was using older .hpa file for testing and it had correct separator. Seems like I accidentally deleted one character in the meantime. Thanks for that @mrwasp, will change that right now.

Tomivix commented 7 years ago

Okay, fixed it @mrwasp. I will wait a little more in case @Aicedosh shows up. Also do you have serviceable laptop just in case something goes wrong? Mine isn't fit for anything besides browsing web unfortunately...

Aicedosh commented 7 years ago

I was just going to report the same bug :) Anyways I have a laptop that might work but it'd be better if someone else took his as well.

Tomivix commented 7 years ago

Okay, then I think I can merge this one safely. Sorry for it being so rough but I'm too sleepy to come up with something better, Please bring your laptop if you can, @Aicedosh. Maybe @mrwasp has one as well?

Tomivix commented 7 years ago

Okay, done. Now I will make .jar file, add it to the project and mark current status as the v0.9.9 release.

mrwasp commented 7 years ago

Yes, I'll take my laptop too.

Tomivix commented 7 years ago

Good. Wait a while and I will add this .jar so that you can test if it works for you guys as well.

Aicedosh commented 7 years ago

I'm afraid that I won't be able to. I have to go in ~5 mins

Tomivix commented 7 years ago

It's here.

mrwasp commented 7 years ago

Ok, works for me, then I go!