RFR: JDK-8048317 - GPIOLEDSample.java has the wrong package name
Riaz A Aimandi
riaz.aimandi at oracle.com
Wed Jul 2 16:18:21 UTC 2014
Hi Jen,
OK. I suggest adding couple of lines of comment on top of the sample class before committing it.
Thanks
- riaz
On Jul 2, 2014, at 11:28 AM, Jen Dority <jen.dority at oracle.com> wrote:
> Hi Riaz,
>
> I took another look at the sample package names. I decided to change it from com.oracle.dio.sample.gpio to dio.gpio. This is more in line with other samples/demos.
>
> Here's the latest webrev: http://cr.openjdk.java.net/~jld/8048317b/
>
> The only change is the package name and path changes to match. Once this is put back, I will update the device io wiki to reflect the new package name.
>
> Thanks,
> Jen
>
> On 7/1/2014 4:32 PM, Riaz A Aimandi wrote:
>> Hi Jen,
>>
>> As there will be multiple samples, you may want to consider renaming the package directory
>> instead of updating the package name in the source.
>>
>> Thanks,
>>
>> - riaz
>>
>> On Jul 1, 2014, at 12:50 PM, Jen Dority <jen.dority at oracle.com> wrote:
>>
>>> issue: https://bugs.openjdk.java.net/browse/JDK-8048317
>>> webrev: http://cr.openjdk.java.net/~jld/8048317/
>>>
>>> Need a trivial review for a package name correction -- "samples" --> "sample".
>>>
>>> Thanks,
>>> Jen
>>>
>
More information about the dio-dev
mailing list