Actions
Architecture #11968
closed
Architecture #11955: Remove most scala compilation warning
Remove "N-uple adaptation" warning
Status:
Released
Priority:
N/A
Assignee:
Category:
Architecture - Code maintenance
Target version:
Fix check:
Regression:
Description
Scala is deprecating the use of adaptation from n-args to tuple, so for example you need to use:
val x: Option[(Int, Int)] = Some((1,2))
This is to remove some ambiguities that can lead to non-expected result in some case.
I'm not sure I really like the result. In some caes, the alternative tuple notation works very well, but not always:
... myLiftXmlElement % (("class", "error")) // bof
... myLiftXmlElement % ("class" -> "error")) // nice!
But in all case, we should have the smallest number of warnings (ideally, 0) with -Xlint option enabled.
Updated by François ARMAND almost 7 years ago
- Status changed from In progress to Pending technical review
- Assignee changed from François ARMAND to Vincent MEMBRÉ
- Pull Request set to https://github.com/Normation/rudder/pull/1828
Updated by Rudder Quality Assistant almost 7 years ago
- Assignee changed from Vincent MEMBRÉ to François ARMAND
Updated by François ARMAND almost 7 years ago
- Status changed from Pending technical review to Pending release
Applied in changeset rudder|d09dc5a5f8bc26c798a0eed498810ae795314dc9.
Updated by Vincent MEMBRÉ almost 7 years ago
- Status changed from Pending release to Released
This bug has been fixed in Rudder 4.3.0~beta1 which was released today.
- 4.3.0~beta1: Announce Changelog
- Download: https://www.rudder-project.org/site/get-rudder/downloads/
Actions