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,
smime.p7s
Description: S/MIME Cryptographic Signature