cgiboudeaux added subscribers: winterz, cgiboudeaux. cgiboudeaux added inline comments.
INLINE COMMENTS > holidayregion.cpp:31 > // Copied from qlocale_data_p.h and qlocale.h > -// Copyright (C) 2014 Digia Plc and/or its subsidiary(-ies). > +// SPDX-FileCopyrightText: 2014 Digia Plc and /or its subsidiary(-ies). > // To later be replaced by OpenCodes or public Qt api I don't think it should be changed here > lunarphase.cpp:6 > > - Copyright (c) 1989, 1993 //krazy:exclude=copyright > - The Regents of the University of California. All rights reserved. > + SPDX-FileCopyrightText: 1989, 1993 The Regents of the University of > California. All rights reserved. > This looks suspicious @winterz ? > holidayparserdriverplan.cpp:6-13 > + SPDX-FileCopyrightText: Preston Brown <pbr...@kde.org> > + SPDX-FileCopyrightText: Reinhold Kainhofer <reinh...@kainhofer.com> > > Portions contributed by > - Peter Littlefield <plit...@sofkin.ca> > - Armin Liebl <lie...@informatik.tu-muenchen.de> > - Efthimios Mavrogeorgiadis <e...@enl.auth.gr> > - Erwin Hugo Achermann <acher...@inf.ethz.ch> > - > - Major rewrite using Bison C++ skeleton: > - Copyright 2010 John Layt <j...@layt.net> > - > - This library is free software; you can redistribute it and/or > - modify it under the terms of the GNU Library General Public > - License as published by the Free Software Foundation; either > - version 2 of the License, or (at your option) any later version. > - > - This library is distributed in the hope that it will be useful, > - but WITHOUT ANY WARRANTY; without even the implied warranty of > - MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the > - GNU Library General Public License for more details. > - > - You should have received a copy of the GNU Library General Public License > - along with this library; see the file COPYING.LIB. If not, write to the > - Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor, > - Boston, MA 02110-1301, USA. > + SPDX-FileCopyrightText: Peter Littlefield <plit...@sofkin.ca> > + SPDX-FileCopyrightText: Armin Liebl > <lie...@informatik.tu-muenchen.de> > + SPDX-FileCopyrightText: Efthimios Mavrogeorgiadis <e...@enl.auth.gr> The original text lacks clarity. Are these file contributors or copyright holders? @winterz > holidayparserdriverplan.cpp:15 > + > + Major rewrite using Bison C++ skeleton by > + SPDX-FileCopyrightText: 2010 John Layt <j...@layt.net> why is "by" needed here? > holidayparserplan.ypp:3-10 > + SPDX-FileCopyrightText: Thomas Driemeyer <tho...@bitrot.de> > > Adapted for use in KOrganizer by > - Preston Brown <pbr...@kde.org> and > - Reinhold Kainhofer <reinh...@kainhofer.com> > + SPDX-FileCopyrightText: Preston Brown <pbr...@kde.org> and > + SPDX-FileCopyrightText: Reinhold Kainhofer <reinh...@kainhofer.com> > > Portions contributed by same question for this file > holidayscannerplan.cpp:1297-1301 > Original version from plan by Thomas Driemeyer <tho...@bitrot.de> > > Adapted for use in KOrganizer by > Preston Brown <pbr...@kde.org> and > Reinhold Kainhofer <reinh...@kainhofer.com> and these lines were not updated > qcalendarsystem.cpp:833 > \legalese > - Copyright (c) 1989 The Regents of the University of California. > + SPDX-FileCopyrightText: 1989 The Regents of the University of California > . > All rights reserved. also looks wrong REPOSITORY R175 KHolidays REVISION DETAIL https://phabricator.kde.org/D27363 To: cordlandwehr Cc: cgiboudeaux, winterz, kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, ngraham, bruns