Small improvments

[ikrivosheev]

PR Info

Changes

• deprecated: OnConflict::update_value, we can now use OnConflict::update_expr
• deprecated: OnConflict::update_values, we can now use OnConflict::update_exprs
• Remove unneeded vec!, we can use array (rust version 1.51)
• Improve OrderedStatement::order_by_customs (instead Vec<_> parameter is generic)
• Improve OrderedStatement::order_by_columns (instead Vec<_> parameter is generic)
• Improve OverStatement::partition_by_customs (instead Vec<_> parameter is generic)
• Improve OverStatement::partition_by_columns (instead Vec<_> parameter is generic)

[billy1624]

Hey @ikrivosheev, nice one!

I always want to do it but haven't start loll Can we also refactor all other vec![]? I believe most of them can be simplify to [] slice. For example:

• sea-query/src/query/condition.rs

  Lines 130 to 138 in c9b9c66

  	/// let query = Query::select()
  	/// .column(Glyph::Image)
  	/// .from(Glyph::Table)
  	/// .cond_where(
  	/// Cond::any()
  	/// .add(Expr::tbl(Glyph::Table, Glyph::Aspect).is_in(vec![3, 4]))
  	/// .add(Expr::tbl(Glyph::Table, Glyph::Image).like("A%"))
  	/// )
  	/// .to_owned();

• vec![3, 4] can be [3, 4]

[tyt2y3]

I felt like I have cleaned up some portion in 1dffb72

[ikrivosheev]

Hey @ikrivosheev, nice one!

I always want to do it but haven't start loll Can we also refactor all other vec![]? I believe most of them can be simplify to [] slice. For example:

* https://github.com/SeaQL/sea-query/blob/c9b9c66af932c6f8f34b63b17ad6242af79a1f0e/src/query/condition.rs#L130-L138

* `vec![3, 4]` can be `[3, 4]`

@billy1624 @tyt2y3 Done! Remove all unneeded vec! and some other improvements! I updated PR description

[billy1624]

Hey @ikrivosheev, thanks for the hard work!! :)

[tyt2y3]

Thank you. This is lots of small improvements indeed.