Skip to content

Commit

Permalink
rust : general remarks
Browse files Browse the repository at this point in the history
  • Loading branch information
dascritch committed Aug 14, 2014
1 parent bce758a commit 79853c0
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 1 deletion.
2 changes: 2 additions & 0 deletions TODO.md
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,8 @@ Noted issues
* Rust is not easy to install up today (version 0.11), but you can use [a repo for ubuntu](http://kianmeng.org/blog/2014/01/20/ubuntu-13-dot-10-rust-installation/)
* Rust do warn on `isValid` , as this language prefer snake instead of camelCase : « warning: function `isValid` should have a snake case name such as `is_valid`, #[warn(non_snake_case_functions)] on by default »
* Yes, messages in Rust are markdown ready ^^
* In Rust, you must prepare regex before, no way to do a dynamic regex. Regex are done at compile time, not after
* Rust is a strictly typed and anticipating compiler : if you declare a function will return a string, your function MUST return a string, whatever the branch scenario is
* Noted (and corrected, that is the exercise goal) typos in documentations :
* [QUnit](https://github.com/jquery/qunitjs.com/pull/77)
* [Atoum](https://github.com/atoum/atoum/pull/345)
Expand Down
2 changes: 1 addition & 1 deletion tests/tests.rs
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ fn encode(b10: &str) -> &str {

match base10 {
Some(number) => b10,
None => "0"
None => { b10 = "0" }
}
return b10
}
Expand Down

0 comments on commit 79853c0

Please sign in to comment.