Skip to content

It's not done, but it's fine - #61

Open
19007361 wants to merge 18 commits into
wvisser:masterfrom
19007361:master
Open

It's not done, but it's fine#61
19007361 wants to merge 18 commits into
wvisser:masterfrom
19007361:master

Conversation

@19007361

Copy link
Copy Markdown

@steynvl

Sorry

@Zhunaidm Zhunaidm left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Fix suggestions.

Comment thread .nfs0000000004601a41000000db Outdated
@@ -0,0 +1,3 @@
Trying location provider `geoclue2'...
Using provider `geoclue2'.
Using method `randr'.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Don't "git add .".

@@ -0,0 +1,172 @@
<?xml version="1.0" encoding="UTF-8" ?>

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Don't push your bin directory. You only want to check in source code into your remote repo.

check(o, "0==0");
}

@Test

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Fix indentation.

Properties props = new Properties();
props.setProperty("green.services", "sat");
props.setProperty("green.service.sat", "(simplify sink)");
//props.setProperty("green.service.sat", "(canonize sink)");

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Remove commented out code.

Expression x = stack.pop();

System.out.println("-------------------");
System.out.println("-------------------");

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Remove debugging code.

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.

2 participants