Merge pull request #12 from bobertoyin/11-change-plottable-traits-plo…

…t-method-to-use-self-instead-of-self feat: change data model plotting to use reference to self instead of …
bobertoyin · May 26, 2022 · 8f8175f · 8f8175f
2 parents 40e98f6 + 762f0ef
commit 8f8175f
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/Cargo.toml b/Cargo.toml
@@ -1,6 +1,6 @@
 [package]
 name = "mbta-rs"
-version = "0.3.0"
+version = "0.4.0"
 edition = "2021"
 authors = ["Robert Yin <bobertoyin@gmail.com>"]
 description = "Simple Rust client for interacting with the MBTA V3 API."

diff --git a/src/map/mod.rs b/src/map/mod.rs
@@ -127,11 +127,11 @@ pub trait Plottable<D> {
     /// * `map` - mutable reference to a tile map
     /// * `anti_alias` - whether to render with anti-aliasing or not
     /// * `plot_style` - plot style for the model
-    fn plot(self, map: &mut StaticMap, anti_alias: bool, extra_data: D) -> Result<(), PlotError>;
+    fn plot(&self, map: &mut StaticMap, anti_alias: bool, extra_data: D) -> Result<(), PlotError>;
 }
 
 impl Plottable<PlotStyle> for Stop {
-    fn plot(self, map: &mut StaticMap, anti_alias: bool, plot_style: PlotStyle) -> Result<(), PlotError> {
+    fn plot(&self, map: &mut StaticMap, anti_alias: bool, plot_style: PlotStyle) -> Result<(), PlotError> {
         if let Some(border_data) = plot_style.border {
             let border = CircleBuilder::new()
                 .lat_coordinate(self.attributes.latitude)
@@ -153,7 +153,7 @@ impl Plottable<PlotStyle> for Stop {
 }
 
 impl Plottable<IconStyle> for Vehicle {
-    fn plot(self, map: &mut StaticMap, _anti_alias: bool, icon_style: IconStyle) -> Result<(), PlotError> {
+    fn plot(&self, map: &mut StaticMap, _anti_alias: bool, icon_style: IconStyle) -> Result<(), PlotError> {
         let icon = IconBuilder::new()
             .lat_coordinate(self.attributes.latitude)
             .lon_coordinate(self.attributes.longitude)
@@ -167,7 +167,7 @@ impl Plottable<IconStyle> for Vehicle {
 }
 
 impl Plottable<PlotStyle> for Shape {
-    fn plot(self, map: &mut StaticMap, anti_alias: bool, plot_style: PlotStyle) -> Result<(), PlotError> {
+    fn plot(&self, map: &mut StaticMap, anti_alias: bool, plot_style: PlotStyle) -> Result<(), PlotError> {
         let points = decode_polyline(&self.attributes.polyline, 5).map_err(PlotError::PolylineError)?;
         if let Some(border_data) = plot_style.border {
             let border = LineBuilder::new()