liszt icon indicating copy to clipboard operation
liszt copied to clipboard

Code repeated heavily throughout project

Open scamacho23 opened this issue 5 years ago • 17 comments

🐛 Describe the bug

Specifically, the issue is that several methods repeat code, so it would be best to decompose out helper functions to use.

scamacho23 avatar Aug 07 '20 18:08 scamacho23

Hi @scamacho23 . Can I work on this issue?

s-surineni avatar Oct 02 '20 05:10 s-surineni

Hi @s-surineni! Absolutely. The help is appreciated!

scamacho23 avatar Oct 02 '20 06:10 scamacho23

Can I work on this too

mishatheking avatar Oct 03 '20 03:10 mishatheking

Hi, I wanted to try liszt. I got getting segmentation fault error. I installed liszt on mac through brew. Am I missing something? image

s-surineni avatar Oct 03 '20 04:10 s-surineni

Hey @mishatheking you can totally work on this as well.

scamacho23 avatar Oct 05 '20 20:10 scamacho23

@s-surineni I'm not sure why that's happening to be honest -- did brew install v1.1.8? I know there was some compatibility issue between 1.1.7 and 1.1.8 with brew. For now, just do the git clone. I think that a merge I previously committed from another contributor may have introduced some breaking measures, so let me try and revert that.

scamacho23 avatar Oct 05 '20 20:10 scamacho23

@anujpatel224 have at it. As a I said above, still not sure why Liszt is breaking on install -- stick to the git clone for now.

scamacho23 avatar Oct 21 '20 22:10 scamacho23

Is this issue still open? If yes, can I work on it?

Shikhar-SRJ avatar May 16 '21 18:05 Shikhar-SRJ

Yes -- there are some weird issues with the build and I haven't gotten around to fixing them yet. I think git cloning the repo should avoid that but let me know if not.

scamacho23 avatar May 16 '21 18:05 scamacho23

Hi, I see this issue is still open. If that's the case, I would like to work on it if possible.

kitsiosvas avatar Jan 25 '22 22:01 kitsiosvas

Absolutely @kitsiosvas -- I haven't worked on this lately so not sure what the build status is. I doubt using brew to install works but cloning the repo should be fine.

scamacho23 avatar Jan 25 '22 22:01 scamacho23

Thank you for your quick reply @scamacho23. I will inspect the code starting tonight even to locate the duplicates across the project. In case there is a list on where the duplicates are, I would highly appreciate it if I could be provided with this information.

kitsiosvas avatar Jan 25 '22 23:01 kitsiosvas

@scamacho23 I've done some changes ( deleted duplicate code by calling a function) in the memory.c file. Where would you want me to commit the code/issue a pull request? The reason I'm asking is that don't want to do something that I shouldn't. Also will have some changes in the util.c file by tonight. Thank you.

kitsiosvas avatar Jan 26 '22 12:01 kitsiosvas

Hi, is this still something I can work on?

rjthompson2 avatar Aug 16 '22 17:08 rjthompson2

@kitsiosvas You fork this repository, commit your changes in your fork and make a Pull Request (PR). See GitHub Docs for more details.

@rjthompson2 probably. ask @scamacho23 to be sure.

Friendly-Banana avatar Sep 26 '22 09:09 Friendly-Banana

@scamacho23 Hello! I am a complete beginner in open source and am just starting out. This is my first commit in other's repository. I have created a pull request.

GuyEternal avatar Oct 01 '22 14:10 GuyEternal

Could I try to resolve this issue if it still hasn't been resolved?

maazshakeel avatar May 25 '23 01:05 maazshakeel