RFR: JDK-8066263 - Split UARTimpl and UARTModemImpl implementation in DIO realization
Sergey Nazarkin
sergey.nazarkin at oracle.com
Thu Dec 4 13:26:47 UTC 2014
This part of implementation internals not covered by the spec. Actually,
it should not be.
But I agree this change is result of inaccurate merge.
/Sergey
On 04.12.2014 16:08, Jen Dority wrote:
> Why this change?
>
> @@ -209,9 +203,6 @@
> }
> // first port in list is DEFAULT port
> String ports =
> Configuration.getProperty("microedition.commports");
> - if (ports == null) {
> - ports = Configuration.getProperty("jdk.dio.uart.ports");
> - }
> if (null != ports) {
> StringTokenizer t = new StringTokenizer(ports, ",");
> while(devNum-- > 0 && t.hasMoreTokens()) {
>
> This had nothing to do with the issue. The microedition.* property
> isn't appropriate for SE. IMO, we should put this back and get get it
> officially sanctioned in the spec.
>
> Jen
>
>
> On 12/4/2014 5:49 AM, Sergey Nazarkin wrote:
>> Looks OK
>>
>> /Sergey
>>
>> On 04.12.2014 13:14, alexey mironov wrote:
>>> new webrev: http://cr.openjdk.java.net/~amironov/8066263.1/
>>>
>>> Previous webrev contain fix for 2 bugs. I prepare new webrev. Please
>>> review it.
>>>
>>> Regards,
>>> Alexey
>>>
>>> On 01.12.2014 18:02, alexey mironov wrote:
>>>> issue: https://bugs.openjdk.java.net/browse/JDK-8066263
>>>> <https://bugs.openjdk.java.net/browse/JDK-8065776>
>>>> webrev: http://cr.openjdk.java.net/~amironov/8066263/
>>>> <https://bugs.openjdk.java.net/browse/JDK-8065776>
>>>>
>>>> Please review the changes made in order to split UART and UARTModem
>>>> implementations.
>>>>
>>>> Regards,
>>>> Alexey
>>>
>>
>
More information about the dio-dev
mailing list