Skip to content

Rewrote "bad" Code to more modern version#108

Open
melvin-suter wants to merge 1 commit intonextcloud:masterfrom
melvin-suter:rewrite_smb_returnval
Open

Rewrote "bad" Code to more modern version#108
melvin-suter wants to merge 1 commit intonextcloud:masterfrom
melvin-suter:rewrite_smb_returnval

Conversation

@melvin-suter
Copy link
Contributor

As discussed in #100 here is the PR with the rewroten if/else statement.

…son with "switch-case"

Signed-off-by: Melvin Suter <github@damon.ch>
Copy link

@ufozone ufozone left a comment

Choose a reason for hiding this comment

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

approved

@violoncelloCH
Copy link
Member

thank you very much for your contribution!
indeed, your changes make sense
however this needs proper testing in different scenarios before merging, but unfortunately I neither have an SMB setup nor enough time to do this at the moment :/

@violoncelloCH violoncelloCH added 3. to review enhancement New feature or request labels Oct 5, 2019
@violoncelloCH
Copy link
Member

@ufozone simply "accepting" the PR doesn't really help here, because I don't know what this "accepting" is based on. Instead please test it as much as possible and explain what you tested in detail :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants