@@ -4,7 +4,7 @@ use std::path::Path;
4
4
use std:: sync:: Mutex ;
5
5
6
6
use crate :: get_las_writer:: get_las_writer;
7
- use crate :: { convert_point:: convert_point, utils:: create_path, LasVersion } ;
7
+ use crate :: { LasVersion , convert_point:: convert_point, utils:: create_path} ;
8
8
9
9
use anyhow:: { Context , Result } ;
10
10
use e57:: { E57Reader , PointCloud } ;
@@ -52,9 +52,7 @@ pub fn convert_pointcloud(
52
52
let point = p. context ( "Could not read point: " ) ?;
53
53
54
54
if point. color . is_some ( ) {
55
- * has_color_mutex
56
- . lock ( )
57
- . unwrap_or_else ( |poisoned| poisoned. into_inner ( ) ) = true ;
55
+ * has_color_mutex. lock ( ) . unwrap_or_else ( |e| e. into_inner ( ) ) = true ;
58
56
}
59
57
60
58
let las_point = match convert_point ( point) {
@@ -77,10 +75,8 @@ pub fn convert_pointcloud(
77
75
)
78
76
. context ( "Unable to create path: " ) ?;
79
77
80
- let has_color = * has_color_mutex
81
- . lock ( )
82
- . unwrap_or_else ( |poisoned| poisoned. into_inner ( ) ) ;
83
-
78
+ let has_color = * has_color_mutex. lock ( ) . unwrap_or_else ( |e| e. into_inner ( ) ) ;
79
+
84
80
let mut writer = get_las_writer (
85
81
pointcloud. clone ( ) . guid ,
86
82
path,
@@ -134,11 +130,9 @@ pub fn convert_pointclouds(
134
130
. par_iter ( )
135
131
. enumerate ( )
136
132
. try_for_each ( |( index, pointcloud) | -> Result < ( ) > {
137
- println ! ( "Saving pointclouds {}..." , index ) ;
133
+ println ! ( "Saving pointclouds {index }..." ) ;
138
134
139
- let mut reader = e57_reader_mutex
140
- . lock ( )
141
- . unwrap_or_else ( |poisoned| poisoned. into_inner ( ) ) ;
135
+ let mut reader = e57_reader_mutex. lock ( ) . unwrap_or_else ( |e| e. into_inner ( ) ) ;
142
136
let pointcloud_reader = reader
143
137
. pointcloud_simple ( pointcloud)
144
138
. context ( "Unable to get point cloud iterator: " ) ?;
@@ -149,9 +143,7 @@ pub fn convert_pointclouds(
149
143
let point = p. context ( "Could not read point: " ) ?;
150
144
151
145
if point. color . is_some ( ) {
152
- * has_color_mutex
153
- . lock ( )
154
- . unwrap_or_else ( |poisoned| poisoned. into_inner ( ) ) = true ;
146
+ * has_color_mutex. lock ( ) . unwrap_or_else ( |e| e. into_inner ( ) ) = true ;
155
147
}
156
148
157
149
let las_point = match convert_point ( point) {
@@ -164,13 +156,13 @@ pub fn convert_pointclouds(
164
156
max_z = max_z. max ( las_point. z ) ;
165
157
las_points_mutex
166
158
. lock ( )
167
- . unwrap_or_else ( |poisoned| poisoned . into_inner ( ) )
159
+ . unwrap_or_else ( |e| e . into_inner ( ) )
168
160
. push ( las_point) ;
169
161
}
170
162
171
163
let mut guard = max_cartesian_mutex
172
164
. lock ( )
173
- . unwrap_or_else ( |poisoned| poisoned . into_inner ( ) ) ;
165
+ . unwrap_or_else ( |e| e . into_inner ( ) ) ;
174
166
let current_max_cartesian = guard. max ( max_x) . max ( max_y) . max ( max_z) ;
175
167
* guard = current_max_cartesian;
176
168
@@ -187,12 +179,10 @@ pub fn convert_pointclouds(
187
179
188
180
let max_cartesian = * max_cartesian_mutex
189
181
. lock ( )
190
- . unwrap_or_else ( |poisoned| poisoned. into_inner ( ) ) ;
191
-
192
- let has_color = * has_color_mutex
193
- . lock ( )
194
- . unwrap_or_else ( |poisoned| poisoned. into_inner ( ) ) ;
195
-
182
+ . unwrap_or_else ( |e| e. into_inner ( ) ) ;
183
+
184
+ let has_color = * has_color_mutex. lock ( ) . unwrap_or_else ( |e| e. into_inner ( ) ) ;
185
+
196
186
let mut writer = get_las_writer (
197
187
Some ( guid. to_owned ( ) ) ,
198
188
path,
@@ -204,9 +194,9 @@ pub fn convert_pointclouds(
204
194
205
195
let las_points = las_points_mutex
206
196
. lock ( )
207
- . unwrap_or_else ( |poisoned| poisoned . into_inner ( ) )
197
+ . unwrap_or_else ( |e| e . into_inner ( ) )
208
198
. clone ( ) ;
209
-
199
+
210
200
for p in las_points {
211
201
writer. write_point ( p) . context ( "Unable to write: " ) ?;
212
202
}
0 commit comments