Skip to content

Commit b1b009b

Browse files
committed
Sanitise _setlonlab function
1 parent 4633f3e commit b1b009b

File tree

1 file changed

+30
-26
lines changed

1 file changed

+30
-26
lines changed

packages/basemap/src/mpl_toolkits/basemap/__init__.py

Lines changed: 30 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -5316,41 +5316,45 @@ def __delitem__(self,key):
53165316
self[key].remove()
53175317
super(_dict, self).__delitem__(key)
53185318

5319-
def _setlonlab(fmt,lon,labelstyle):
5320-
# set lon label string (called by Basemap.drawmeridians)
5321-
try: # fmt is a function that returns a formatted string
5319+
5320+
def _setlonlab(fmt, lon, labelstyle):
5321+
"""Set longitude label string (called by :meth:`Basemap.drawmeridians`)."""
5322+
5323+
try:
5324+
# `fmt` is a function that returns a formatted string.
53225325
lonlab = fmt(lon)
5323-
except: # fmt is a format string.
5324-
if lon>180:
5325-
if mpl.rcParams['text.usetex']:
5326-
if labelstyle=='+/-':
5327-
lonlabstr = r'${\/-%s\/^{\circ}}$'%fmt
5326+
except:
5327+
# `fmt` is a format string.
5328+
if lon > 180:
5329+
if mpl.rcParams["text.usetex"]:
5330+
if labelstyle == "+/-":
5331+
lonlabstr = r"${\/-%s\/^{\circ}}$" % fmt
53285332
else:
5329-
lonlabstr = r'${%s\/^{\circ}\/W}$'%fmt
5333+
lonlabstr = r"${%s\/^{\circ}\/W}$" % fmt
53305334
else:
5331-
if labelstyle=='+/-':
5332-
lonlabstr = u'-%s\N{DEGREE SIGN}'%fmt
5335+
if labelstyle == "+/-":
5336+
lonlabstr = u"-%s\N{DEGREE SIGN}" % fmt
53335337
else:
5334-
lonlabstr = u'%s\N{DEGREE SIGN}W'%fmt
5335-
lonlab = lonlabstr%np.fabs(lon-360)
5336-
elif lon<180 and lon != 0:
5337-
if mpl.rcParams['text.usetex']:
5338-
if labelstyle=='+/-':
5339-
lonlabstr = r'${\/+%s\/^{\circ}}$'%fmt
5338+
lonlabstr = u"%s\N{DEGREE SIGN}W" % fmt
5339+
lonlab = lonlabstr % np.fabs(lon - 360)
5340+
elif lon < 180 and lon != 0:
5341+
if mpl.rcParams["text.usetex"]:
5342+
if labelstyle == "+/-":
5343+
lonlabstr = r"${\/+%s\/^{\circ}}$" % fmt
53405344
else:
5341-
lonlabstr = r'${%s\/^{\circ}\/E}$'%fmt
5345+
lonlabstr = r"${%s\/^{\circ}\/E}$" % fmt
53425346
else:
5343-
if labelstyle=='+/-':
5344-
lonlabstr = u'+%s\N{DEGREE SIGN}'%fmt
5347+
if labelstyle == "+/-":
5348+
lonlabstr = u"+%s\N{DEGREE SIGN}" % fmt
53455349
else:
5346-
lonlabstr = u'%s\N{DEGREE SIGN}E'%fmt
5347-
lonlab = lonlabstr%lon
5350+
lonlabstr = u"%s\N{DEGREE SIGN}E" % fmt
5351+
lonlab = lonlabstr % lon
53485352
else:
5349-
if mpl.rcParams['text.usetex']:
5350-
lonlabstr = r'${%s\/^{\circ}}$'%fmt
5353+
if mpl.rcParams["text.usetex"]:
5354+
lonlabstr = r"${%s\/^{\circ}}$" % fmt
53515355
else:
5352-
lonlabstr = u'%s\N{DEGREE SIGN}'%fmt
5353-
lonlab = lonlabstr%lon
5356+
lonlabstr = u"%s\N{DEGREE SIGN}" % fmt
5357+
lonlab = lonlabstr % lon
53545358
return lonlab
53555359

53565360

0 commit comments

Comments
 (0)