Skip to content

Fix cipher#51

Open
KATESTYLES333 wants to merge 4 commits intosecourse2019:masterfrom
KATESTYLES333:fixCipher
Open

Fix cipher#51
KATESTYLES333 wants to merge 4 commits intosecourse2019:masterfrom
KATESTYLES333:fixCipher

Conversation

@KATESTYLES333
Copy link
Collaborator

No description provided.

@ghost
Copy link

ghost commented Nov 1, 2019

The JavaSee analysis for the commit 6c232b7 failed. ⛑
Visit the analysis result or the error log for more details.

@ghost
Copy link

ghost commented Nov 1, 2019

Checkstyle detected 108 issues on the commit 6c232b7.
Visit the analysis result to review all issues.

TableCipherMethod(String message, int value){};

public StreamMessage encryptor() {
private void ChangeInstanse(int value) {
Copy link

Choose a reason for hiding this comment

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

[PMD] Avoid unused method parameters such as 'value'. (see on Sider)

Details

References:

}

public StreamMessage decryptor() {
private int rolling(int value, int index) {
Copy link

Choose a reason for hiding this comment

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

[PMD] Avoid unused method parameters such as 'index'. (see on Sider)

Details

References:

}

public StreamMessage decryptor() {
private int rolling(int value, int index) {
Copy link

Choose a reason for hiding this comment

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

[PMD] Avoid unused private methods such as 'rolling(int,int)'. (see on Sider)

Details

References:

TableCipherMethod(String message, int value){};

public StreamMessage encryptor() {
private void ChangeInstanse(int value) {
Copy link

Choose a reason for hiding this comment

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

[PMD] Avoid unused private methods such as 'ChangeInstanse(int)'. (see on Sider)

Details

References:

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

Successfully merging this pull request may close these issues.

1 participant