Date: Fri, 26 Jun 2020 21:33:23 -0600 From: Warner Losh <imp@bsdimp.com> To: Enji Cooper <yaneurabeya@gmail.com> Cc: Alexey Dokuchaev <danfe@freebsd.org>, Matt Macy <mmacy@freebsd.org>, src-committers <src-committers@freebsd.org>, svn-src-all <svn-src-all@freebsd.org>, svn-src-head <svn-src-head@freebsd.org> Subject: Re: svn commit: r362667 - head/sys/cddl/contrib/opensolaris/uts/common/dtrace Message-ID: <CANCZdfrr3P4pYeL1EQ50gbZbtJ-jJ-XUPftitb1A%2BSC63dnjRw@mail.gmail.com> In-Reply-To: <94C5CFE0-2F25-44FA-B0B7-D6397D4EAC7A@gmail.com> References: <202006270057.05R0vmgC070914@repo.freebsd.org> <20200627010206.GA63841@FreeBSD.org> <94C5CFE0-2F25-44FA-B0B7-D6397D4EAC7A@gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On Fri, Jun 26, 2020, 9:04 PM Enji Cooper <yaneurabeya@gmail.com> wrote: > > > On Jun 26, 2020, at 6:02 PM, Alexey Dokuchaev <danfe@FreeBSD.org> wrote= : > > > > On Sat, Jun 27, 2020 at 12:57:48AM +0000, Matt Macy wrote: > >> New Revision: 362667 > >> URL: https://svnweb.freebsd.org/changeset/base/362667 > >> > >> Log: > >> Fix "current" variable name conflict with openzfs > >> > >> The variable "current" is an alias for curthread > >> in openzfs. Rename all variable uses of current > >> in dtrace.c to curstate. > > > > Shouldn't OpenZFS be fixed instead? The fact that we need to fix our > > local variables suggests that they're abusing too generic name=E2=80=A6 > > > I kind of agree with `current` being a bit too genericly named in > both DTrace and OpenZFS :/.. > It's always going to be something. These changes are actually pretty minor and really only affect the part of the tree that uses Open Solaris compatibility layer. These changes are smaller for us than changing all OpenZFS. This really was the right fix. Warner >
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CANCZdfrr3P4pYeL1EQ50gbZbtJ-jJ-XUPftitb1A%2BSC63dnjRw>