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