change impl to support json path on generic value#4
change impl to support json path on generic value#4MeirShpilraien wants to merge 30 commits intomasterfrom
Conversation
Co-authored-by: Omer Shadmi <76992134+oshadmi@users.noreply.github.com>
Slice tokenizer
This reverts commit 4b11540. Decided this api does not pertain to this trait
Changes needed for ReJSON multi values and multi paths #477
* Avoid panic on unimplemeted filter syntax * Unimplemeted filter syntax - add unit test * Unimplemeted filter syntax - add unit test (2) cargo test unimplemented_in_filter
ijson support
* avoid panic on bad filter * Formatting * return empty filter token instead of panic * add test
![sonatype-lift[bot]](https://avatars.githubusercontent.com/in/9843?s=60&v=4){kind=small}
| Value::Array(_) => SelectValueType::Array, | ||
| Value::Object(_) => SelectValueType::Object, | ||
| Value::Number(n) => { | ||
| if n.is_i64() { |
There was a problem hiding this comment.
compiler-message: error: this if has identical blocks
--> src/select/json_node.rs:16:31
|
16 | if n.is_i64() {
| _______________________________^
17 | | SelectValueType::Long
18 | | } else if n.is_u64() {
| |_________________^
|
= note: #[deny(clippy::if_same_then_else)] on by default
note: same as this
--> src/select/json_node.rs:18:38
|
18 | } else if n.is_u64() {
| _____________________^
19 | | SelectValueType::Long
20 | | } else if n.is_f64() {
| |^
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#if_same_then_else
(at-me in a reply with help or ignore)
Break after the first equals result_value to avoid duplicates
3 participants
No description provided.