You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@openoffice.apache.org by Dick Groskamp <di...@apache.org> on 2021/03/04 18:25:10 UTC
Is this a typo or is it intentional?
I'm definitely no developer but when I added the new Japanse era Reiwa to the file
openoffice-trunk/main/i18npool/source/calendar/calendar_gregorian.cxx,
I noticed this code:
________________________________________
132 Calendar_gregorian::Calendar_gregorian()
133 {
134 init(NULL);
135 }
136 Calendar_gregorian::Calendar_gregorian(Era *_earArray)
137 {
138 init(_earArray);
139 }
_________________________________________
on line 140 it says however: Calendar_gregorian::init(Era *_eraArray)
It seems to me that _eraArray is right because it is used further on in the code.
QUESTION:
Are the *_earArray on line 136 and _earArray on line 138 intentional or are they typo's ?
Could someone with more knowledge enlighten me or at least check if it is correct as it is ?
Couldn't file an issue on GitHub so mentioned it here
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@openoffice.apache.org
For additional commands, e-mail: dev-help@openoffice.apache.org
Re: <code> Is this a typo or is it intentional?
Posted by Matthias Seidel <ma...@hamburg.de>.
Hi Dick,
I just created a Pull Request for it and asked Arrigo for review:
https://github.com/apache/openoffice/pull/123
Regards,
Matthias
Am 05.03.21 um 12:33 schrieb Dick Groskamp:
> Rigo / Matthias,
>
> thanks for the info. Now I know that I saw it right then.
> I'm not touching the code as I have no idea how to fix it.
>
> On 2021/03/05 09:51:58, Matthias Seidel <ma...@hamburg.de> wrote:
>> Hi all,
>>
>> It really seems to be a typo and OpenGrok can only find two instances of it:
>>
>> http://opengrok.openoffice.org/search?project=trunk&full=%22_earArray%22&defs=&refs=&path=&hist=&type=&si=full
>>
>> So I think it would be without risk to correct this.
>>
>> Regards,
>>
>> Matthias
>>
>> Am 05.03.21 um 08:31 schrieb Arrigo Marchiori:
>>> Hello Dick,
>>>
>>> On Thu, Mar 04, 2021 at 06:25:10PM -0000, Dick Groskamp wrote:
>>>
>>>> I'm definitely no developer but when I added the new Japanse era Reiwa to the file
>>>> openoffice-trunk/main/i18npool/source/calendar/calendar_gregorian.cxx,
>>>> I noticed this code:
>>>> ________________________________________
>>>> 132 Calendar_gregorian::Calendar_gregorian()
>>>> 133 {
>>>> 134 init(NULL);
>>>> 135 }
>>>> 136 Calendar_gregorian::Calendar_gregorian(Era *_earArray)
>>>> 137 {
>>>> 138 init(_earArray);
>>>> 139 }
>>>> _________________________________________
>>>> on line 140 it says however: Calendar_gregorian::init(Era *_eraArray)
>>>>
>>>> It seems to me that _eraArray is right because it is used further on in the code.
>>>>
>>>> QUESTION:
>>>> Are the *_earArray on line 136 and _earArray on line 138 intentional or are they typo's ?
>>> I think that "ear" instead of "era" is a typo.
>>>
>>> If you look at the corresponding include file
>>> main/i18npool/inc/calendar_gregorian.hxx at line 51:
>>>
>>> 45 class Calendar_gregorian : public CalendarImpl
>>> 46 {
>>> 47 public:
>>> 48
>>> 49 // Constructors
>>> 50 Calendar_gregorian();
>>> 51 Calendar_gregorian(Era *_eraArray);
>>> 52 void SAL_CALL init(Era *_eraArray);
>>>
>>> The parameter is spelled correctly there.
>>>
>>> However the code compiles and works fine, because the typo is ``used
>>> consistently'' inside Calendar_gregorian::Calendar_gregorian(Era *)
>>>
>>> If you don't like to have that typo in the code, I believe you can fix
>>> it straight away as it is a ``trivial edit'' (i.e. no need to review
>>> before committing), as I think you did for the new Japanese era.
>>>
>>> I hope this helps.
>>>
>>> Best regards,
>>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@openoffice.apache.org
> For additional commands, e-mail: dev-help@openoffice.apache.org
>
>
Re: <code> Is this a typo or is it intentional?
Posted by Dick Groskamp <di...@apache.org>.
Rigo / Matthias,
thanks for the info. Now I know that I saw it right then.
I'm not touching the code as I have no idea how to fix it.
On 2021/03/05 09:51:58, Matthias Seidel <ma...@hamburg.de> wrote:
> Hi all,
>
> It really seems to be a typo and OpenGrok can only find two instances of it:
>
> http://opengrok.openoffice.org/search?project=trunk&full=%22_earArray%22&defs=&refs=&path=&hist=&type=&si=full
>
> So I think it would be without risk to correct this.
>
> Regards,
>
> Matthias
>
> Am 05.03.21 um 08:31 schrieb Arrigo Marchiori:
> > Hello Dick,
> >
> > On Thu, Mar 04, 2021 at 06:25:10PM -0000, Dick Groskamp wrote:
> >
> >> I'm definitely no developer but when I added the new Japanse era Reiwa to the file
> >> openoffice-trunk/main/i18npool/source/calendar/calendar_gregorian.cxx,
> >> I noticed this code:
> >> ________________________________________
> >> 132 Calendar_gregorian::Calendar_gregorian()
> >> 133 {
> >> 134 init(NULL);
> >> 135 }
> >> 136 Calendar_gregorian::Calendar_gregorian(Era *_earArray)
> >> 137 {
> >> 138 init(_earArray);
> >> 139 }
> >> _________________________________________
> >> on line 140 it says however: Calendar_gregorian::init(Era *_eraArray)
> >>
> >> It seems to me that _eraArray is right because it is used further on in the code.
> >>
> >> QUESTION:
> >> Are the *_earArray on line 136 and _earArray on line 138 intentional or are they typo's ?
> > I think that "ear" instead of "era" is a typo.
> >
> > If you look at the corresponding include file
> > main/i18npool/inc/calendar_gregorian.hxx at line 51:
> >
> > 45 class Calendar_gregorian : public CalendarImpl
> > 46 {
> > 47 public:
> > 48
> > 49 // Constructors
> > 50 Calendar_gregorian();
> > 51 Calendar_gregorian(Era *_eraArray);
> > 52 void SAL_CALL init(Era *_eraArray);
> >
> > The parameter is spelled correctly there.
> >
> > However the code compiles and works fine, because the typo is ``used
> > consistently'' inside Calendar_gregorian::Calendar_gregorian(Era *)
> >
> > If you don't like to have that typo in the code, I believe you can fix
> > it straight away as it is a ``trivial edit'' (i.e. no need to review
> > before committing), as I think you did for the new Japanese era.
> >
> > I hope this helps.
> >
> > Best regards,
>
>
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@openoffice.apache.org
For additional commands, e-mail: dev-help@openoffice.apache.org
Re: Is this a typo or is it intentional?
Posted by Matthias Seidel <ma...@hamburg.de>.
Hi all,
It really seems to be a typo and OpenGrok can only find two instances of it:
http://opengrok.openoffice.org/search?project=trunk&full=%22_earArray%22&defs=&refs=&path=&hist=&type=&si=full
So I think it would be without risk to correct this.
Regards,
Matthias
Am 05.03.21 um 08:31 schrieb Arrigo Marchiori:
> Hello Dick,
>
> On Thu, Mar 04, 2021 at 06:25:10PM -0000, Dick Groskamp wrote:
>
>> I'm definitely no developer but when I added the new Japanse era Reiwa to the file
>> openoffice-trunk/main/i18npool/source/calendar/calendar_gregorian.cxx,
>> I noticed this code:
>> ________________________________________
>> 132 Calendar_gregorian::Calendar_gregorian()
>> 133 {
>> 134 init(NULL);
>> 135 }
>> 136 Calendar_gregorian::Calendar_gregorian(Era *_earArray)
>> 137 {
>> 138 init(_earArray);
>> 139 }
>> _________________________________________
>> on line 140 it says however: Calendar_gregorian::init(Era *_eraArray)
>>
>> It seems to me that _eraArray is right because it is used further on in the code.
>>
>> QUESTION:
>> Are the *_earArray on line 136 and _earArray on line 138 intentional or are they typo's ?
> I think that "ear" instead of "era" is a typo.
>
> If you look at the corresponding include file
> main/i18npool/inc/calendar_gregorian.hxx at line 51:
>
> 45 class Calendar_gregorian : public CalendarImpl
> 46 {
> 47 public:
> 48
> 49 // Constructors
> 50 Calendar_gregorian();
> 51 Calendar_gregorian(Era *_eraArray);
> 52 void SAL_CALL init(Era *_eraArray);
>
> The parameter is spelled correctly there.
>
> However the code compiles and works fine, because the typo is ``used
> consistently'' inside Calendar_gregorian::Calendar_gregorian(Era *)
>
> If you don't like to have that typo in the code, I believe you can fix
> it straight away as it is a ``trivial edit'' (i.e. no need to review
> before committing), as I think you did for the new Japanese era.
>
> I hope this helps.
>
> Best regards,
Re: Is this a typo or is it intentional?
Posted by Arrigo Marchiori <ar...@yahoo.it.INVALID>.
Hello Dick,
On Thu, Mar 04, 2021 at 06:25:10PM -0000, Dick Groskamp wrote:
> I'm definitely no developer but when I added the new Japanse era Reiwa to the file
> openoffice-trunk/main/i18npool/source/calendar/calendar_gregorian.cxx,
> I noticed this code:
> ________________________________________
> 132 Calendar_gregorian::Calendar_gregorian()
> 133 {
> 134 init(NULL);
> 135 }
> 136 Calendar_gregorian::Calendar_gregorian(Era *_earArray)
> 137 {
> 138 init(_earArray);
> 139 }
> _________________________________________
> on line 140 it says however: Calendar_gregorian::init(Era *_eraArray)
>
> It seems to me that _eraArray is right because it is used further on in the code.
>
> QUESTION:
> Are the *_earArray on line 136 and _earArray on line 138 intentional or are they typo's ?
I think that "ear" instead of "era" is a typo.
If you look at the corresponding include file
main/i18npool/inc/calendar_gregorian.hxx at line 51:
45 class Calendar_gregorian : public CalendarImpl
46 {
47 public:
48
49 // Constructors
50 Calendar_gregorian();
51 Calendar_gregorian(Era *_eraArray);
52 void SAL_CALL init(Era *_eraArray);
The parameter is spelled correctly there.
However the code compiles and works fine, because the typo is ``used
consistently'' inside Calendar_gregorian::Calendar_gregorian(Era *)
If you don't like to have that typo in the code, I believe you can fix
it straight away as it is a ``trivial edit'' (i.e. no need to review
before committing), as I think you did for the new Japanese era.
I hope this helps.
Best regards,
--
Arrigo
http://rigo.altervista.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@openoffice.apache.org
For additional commands, e-mail: dev-help@openoffice.apache.org