]> git.lizzy.rs Git - rust.git/blob - src/librustdoc/core.rs
Rollup merge of #86776 - tmiasko:v0-skip-layout-query, r=petrochenkov
[rust.git] / src / librustdoc / core.rs
1 use rustc_data_structures::fx::{FxHashMap, FxHashSet};
2 use rustc_data_structures::sync::{self, Lrc};
3 use rustc_driver::abort_on_err;
4 use rustc_errors::emitter::{Emitter, EmitterWriter};
5 use rustc_errors::json::JsonEmitter;
6 use rustc_feature::UnstableFeatures;
7 use rustc_hir::def::Namespace::TypeNS;
8 use rustc_hir::def::Res;
9 use rustc_hir::def_id::{DefId, LocalDefId, CRATE_DEF_INDEX};
10 use rustc_hir::HirId;
11 use rustc_hir::{
12     intravisit::{self, NestedVisitorMap, Visitor},
13     Path,
14 };
15 use rustc_interface::{interface, Queries};
16 use rustc_middle::hir::map::Map;
17 use rustc_middle::middle::privacy::AccessLevels;
18 use rustc_middle::ty::{ParamEnv, Ty, TyCtxt};
19 use rustc_resolve as resolve;
20 use rustc_session::config::{self, CrateType, ErrorOutputType};
21 use rustc_session::lint;
22 use rustc_session::DiagnosticOutput;
23 use rustc_session::Session;
24 use rustc_span::source_map;
25 use rustc_span::symbol::sym;
26 use rustc_span::{Span, DUMMY_SP};
27
28 use std::cell::RefCell;
29 use std::mem;
30 use std::rc::Rc;
31
32 use crate::clean::inline::build_external_trait;
33 use crate::clean::{self, FakeDefId, TraitWithExtraInfo};
34 use crate::config::{Options as RustdocOptions, OutputFormat, RenderOptions};
35 use crate::formats::cache::Cache;
36 use crate::passes::{self, Condition::*, ConditionalPass};
37
38 crate use rustc_session::config::{DebuggingOptions, Input, Options};
39
40 crate struct DocContext<'tcx> {
41     crate tcx: TyCtxt<'tcx>,
42     /// Name resolver. Used for intra-doc links.
43     ///
44     /// The `Rc<RefCell<...>>` wrapping is needed because that is what's returned by
45     /// [`Queries::expansion()`].
46     // FIXME: see if we can get rid of this RefCell somehow
47     crate resolver: Rc<RefCell<interface::BoxedResolver>>,
48     /// Used for normalization.
49     ///
50     /// Most of this logic is copied from rustc_lint::late.
51     crate param_env: ParamEnv<'tcx>,
52     /// Later on moved through `clean::Crate` into `cache`
53     crate external_traits: Rc<RefCell<FxHashMap<DefId, clean::TraitWithExtraInfo>>>,
54     /// Used while populating `external_traits` to ensure we don't process the same trait twice at
55     /// the same time.
56     crate active_extern_traits: FxHashSet<DefId>,
57     // The current set of type and lifetime substitutions,
58     // for expanding type aliases at the HIR level:
59     /// Table `DefId` of type parameter -> substituted type
60     crate ty_substs: FxHashMap<DefId, clean::Type>,
61     /// Table `DefId` of lifetime parameter -> substituted lifetime
62     crate lt_substs: FxHashMap<DefId, clean::Lifetime>,
63     /// Table `DefId` of const parameter -> substituted const
64     crate ct_substs: FxHashMap<DefId, clean::Constant>,
65     /// Table synthetic type parameter for `impl Trait` in argument position -> bounds
66     crate impl_trait_bounds: FxHashMap<ImplTraitParam, Vec<clean::GenericBound>>,
67     /// Auto-trait or blanket impls processed so far, as `(self_ty, trait_def_id)`.
68     // FIXME(eddyb) make this a `ty::TraitRef<'tcx>` set.
69     crate generated_synthetics: FxHashSet<(Ty<'tcx>, DefId)>,
70     crate auto_traits: Vec<DefId>,
71     /// The options given to rustdoc that could be relevant to a pass.
72     crate render_options: RenderOptions,
73     /// The traits in scope for a given module.
74     ///
75     /// See `collect_intra_doc_links::traits_implemented_by` for more details.
76     /// `map<module, set<trait>>`
77     crate module_trait_cache: FxHashMap<DefId, FxHashSet<DefId>>,
78     /// This same cache is used throughout rustdoc, including in [`crate::html::render`].
79     crate cache: Cache,
80     /// Used by [`clean::inline`] to tell if an item has already been inlined.
81     crate inlined: FxHashSet<FakeDefId>,
82     /// Used by `calculate_doc_coverage`.
83     crate output_format: OutputFormat,
84 }
85
86 impl<'tcx> DocContext<'tcx> {
87     crate fn sess(&self) -> &'tcx Session {
88         &self.tcx.sess
89     }
90
91     crate fn with_param_env<T, F: FnOnce(&mut Self) -> T>(&mut self, def_id: DefId, f: F) -> T {
92         let old_param_env = mem::replace(&mut self.param_env, self.tcx.param_env(def_id));
93         let ret = f(self);
94         self.param_env = old_param_env;
95         ret
96     }
97
98     crate fn enter_resolver<F, R>(&self, f: F) -> R
99     where
100         F: FnOnce(&mut resolve::Resolver<'_>) -> R,
101     {
102         self.resolver.borrow_mut().access(f)
103     }
104
105     /// Call the closure with the given parameters set as
106     /// the substitutions for a type alias' RHS.
107     crate fn enter_alias<F, R>(
108         &mut self,
109         ty_substs: FxHashMap<DefId, clean::Type>,
110         lt_substs: FxHashMap<DefId, clean::Lifetime>,
111         ct_substs: FxHashMap<DefId, clean::Constant>,
112         f: F,
113     ) -> R
114     where
115         F: FnOnce(&mut Self) -> R,
116     {
117         let (old_tys, old_lts, old_cts) = (
118             mem::replace(&mut self.ty_substs, ty_substs),
119             mem::replace(&mut self.lt_substs, lt_substs),
120             mem::replace(&mut self.ct_substs, ct_substs),
121         );
122         let r = f(self);
123         self.ty_substs = old_tys;
124         self.lt_substs = old_lts;
125         self.ct_substs = old_cts;
126         r
127     }
128
129     /// Like `hir().local_def_id_to_hir_id()`, but skips calling it on fake DefIds.
130     /// (This avoids a slice-index-out-of-bounds panic.)
131     crate fn as_local_hir_id(tcx: TyCtxt<'_>, def_id: FakeDefId) -> Option<HirId> {
132         match def_id {
133             FakeDefId::Real(real_id) => {
134                 real_id.as_local().map(|def_id| tcx.hir().local_def_id_to_hir_id(def_id))
135             }
136             FakeDefId::Fake(_, _) => None,
137         }
138     }
139 }
140
141 /// Creates a new diagnostic `Handler` that can be used to emit warnings and errors.
142 ///
143 /// If the given `error_format` is `ErrorOutputType::Json` and no `SourceMap` is given, a new one
144 /// will be created for the handler.
145 crate fn new_handler(
146     error_format: ErrorOutputType,
147     source_map: Option<Lrc<source_map::SourceMap>>,
148     debugging_opts: &DebuggingOptions,
149 ) -> rustc_errors::Handler {
150     let emitter: Box<dyn Emitter + sync::Send> = match error_format {
151         ErrorOutputType::HumanReadable(kind) => {
152             let (short, color_config) = kind.unzip();
153             Box::new(
154                 EmitterWriter::stderr(
155                     color_config,
156                     source_map.map(|sm| sm as _),
157                     short,
158                     debugging_opts.teach,
159                     debugging_opts.terminal_width,
160                     false,
161                 )
162                 .ui_testing(debugging_opts.ui_testing),
163             )
164         }
165         ErrorOutputType::Json { pretty, json_rendered } => {
166             let source_map = source_map.unwrap_or_else(|| {
167                 Lrc::new(source_map::SourceMap::new(source_map::FilePathMapping::empty()))
168             });
169             Box::new(
170                 JsonEmitter::stderr(
171                     None,
172                     source_map,
173                     pretty,
174                     json_rendered,
175                     debugging_opts.terminal_width,
176                     false,
177                 )
178                 .ui_testing(debugging_opts.ui_testing),
179             )
180         }
181     };
182
183     rustc_errors::Handler::with_emitter_and_flags(
184         emitter,
185         debugging_opts.diagnostic_handler_flags(true),
186     )
187 }
188
189 /// Parse, resolve, and typecheck the given crate.
190 crate fn create_config(
191     RustdocOptions {
192         input,
193         crate_name,
194         proc_macro_crate,
195         error_format,
196         libs,
197         externs,
198         mut cfgs,
199         codegen_options,
200         debugging_opts,
201         target,
202         edition,
203         maybe_sysroot,
204         lint_opts,
205         describe_lints,
206         lint_cap,
207         display_warnings,
208         ..
209     }: RustdocOptions,
210 ) -> rustc_interface::Config {
211     // Add the doc cfg into the doc build.
212     cfgs.push("doc".to_string());
213
214     let cpath = Some(input.clone());
215     let input = Input::File(input);
216
217     // By default, rustdoc ignores all lints.
218     // Specifically unblock lints relevant to documentation or the lint machinery itself.
219     let mut lints_to_show = vec![
220         // it's unclear whether these should be part of rustdoc directly (#77364)
221         rustc_lint::builtin::MISSING_DOCS.name.to_string(),
222         rustc_lint::builtin::INVALID_DOC_ATTRIBUTES.name.to_string(),
223         // these are definitely not part of rustdoc, but we want to warn on them anyway.
224         rustc_lint::builtin::RENAMED_AND_REMOVED_LINTS.name.to_string(),
225         rustc_lint::builtin::UNKNOWN_LINTS.name.to_string(),
226     ];
227     lints_to_show.extend(crate::lint::RUSTDOC_LINTS.iter().map(|lint| lint.name.to_string()));
228
229     let (lint_opts, lint_caps) = crate::lint::init_lints(lints_to_show, lint_opts, |lint| {
230         Some((lint.name_lower(), lint::Allow))
231     });
232
233     let crate_types =
234         if proc_macro_crate { vec![CrateType::ProcMacro] } else { vec![CrateType::Rlib] };
235     // plays with error output here!
236     let sessopts = config::Options {
237         maybe_sysroot,
238         search_paths: libs,
239         crate_types,
240         lint_opts: if !display_warnings { lint_opts } else { vec![] },
241         lint_cap,
242         cg: codegen_options,
243         externs,
244         target_triple: target,
245         unstable_features: UnstableFeatures::from_environment(crate_name.as_deref()),
246         actually_rustdoc: true,
247         debugging_opts,
248         error_format,
249         edition,
250         describe_lints,
251         crate_name,
252         ..Options::default()
253     };
254
255     interface::Config {
256         opts: sessopts,
257         crate_cfg: interface::parse_cfgspecs(cfgs),
258         input,
259         input_path: cpath,
260         output_file: None,
261         output_dir: None,
262         file_loader: None,
263         diagnostic_output: DiagnosticOutput::Default,
264         stderr: None,
265         lint_caps,
266         parse_sess_created: None,
267         register_lints: Some(box crate::lint::register_lints),
268         override_queries: Some(|_sess, providers, _external_providers| {
269             // Most lints will require typechecking, so just don't run them.
270             providers.lint_mod = |_, _| {};
271             // Prevent `rustc_typeck::check_crate` from calling `typeck` on all bodies.
272             providers.typeck_item_bodies = |_, _| {};
273             // hack so that `used_trait_imports` won't try to call typeck
274             providers.used_trait_imports = |_, _| {
275                 lazy_static! {
276                     static ref EMPTY_SET: FxHashSet<LocalDefId> = FxHashSet::default();
277                 }
278                 &EMPTY_SET
279             };
280             // In case typeck does end up being called, don't ICE in case there were name resolution errors
281             providers.typeck = move |tcx, def_id| {
282                 // Closures' tables come from their outermost function,
283                 // as they are part of the same "inference environment".
284                 // This avoids emitting errors for the parent twice (see similar code in `typeck_with_fallback`)
285                 let outer_def_id = tcx.closure_base_def_id(def_id.to_def_id()).expect_local();
286                 if outer_def_id != def_id {
287                     return tcx.typeck(outer_def_id);
288                 }
289
290                 let hir = tcx.hir();
291                 let body = hir.body(hir.body_owned_by(hir.local_def_id_to_hir_id(def_id)));
292                 debug!("visiting body for {:?}", def_id);
293                 EmitIgnoredResolutionErrors::new(tcx).visit_body(body);
294                 (rustc_interface::DEFAULT_QUERY_PROVIDERS.typeck)(tcx, def_id)
295             };
296         }),
297         make_codegen_backend: None,
298         registry: rustc_driver::diagnostics_registry(),
299     }
300 }
301
302 crate fn create_resolver<'a>(
303     externs: config::Externs,
304     queries: &Queries<'a>,
305     sess: &Session,
306 ) -> Rc<RefCell<interface::BoxedResolver>> {
307     let extern_names: Vec<String> = externs
308         .iter()
309         .filter(|(_, entry)| entry.add_prelude)
310         .map(|(name, _)| name)
311         .cloned()
312         .collect();
313
314     let (_, resolver, _) = &*abort_on_err(queries.expansion(), sess).peek();
315
316     // Before we actually clone it, let's force all the extern'd crates to
317     // actually be loaded, just in case they're only referred to inside
318     // intra-doc links
319     resolver.borrow_mut().access(|resolver| {
320         sess.time("load_extern_crates", || {
321             for extern_name in &extern_names {
322                 debug!("loading extern crate {}", extern_name);
323                 if let Err(()) = resolver
324                     .resolve_str_path_error(
325                         DUMMY_SP,
326                         extern_name,
327                         TypeNS,
328                         LocalDefId { local_def_index: CRATE_DEF_INDEX }.to_def_id(),
329                   ) {
330                     warn!("unable to resolve external crate {} (do you have an unused `--extern` crate?)", extern_name)
331                   }
332             }
333         });
334     });
335
336     // Now we're good to clone the resolver because everything should be loaded
337     resolver.clone()
338 }
339
340 crate fn run_global_ctxt(
341     tcx: TyCtxt<'_>,
342     resolver: Rc<RefCell<interface::BoxedResolver>>,
343     mut default_passes: passes::DefaultPassOption,
344     manual_passes: Vec<String>,
345     render_options: RenderOptions,
346     output_format: OutputFormat,
347 ) -> (clean::Crate, RenderOptions, Cache) {
348     // Certain queries assume that some checks were run elsewhere
349     // (see https://github.com/rust-lang/rust/pull/73566#issuecomment-656954425),
350     // so type-check everything other than function bodies in this crate before running lints.
351
352     // NOTE: this does not call `tcx.analysis()` so that we won't
353     // typeck function bodies or run the default rustc lints.
354     // (see `override_queries` in the `config`)
355
356     // HACK(jynelson) this calls an _extremely_ limited subset of `typeck`
357     // and might break if queries change their assumptions in the future.
358
359     // NOTE: This is copy/pasted from typeck/lib.rs and should be kept in sync with those changes.
360     tcx.sess.time("item_types_checking", || {
361         for &module in tcx.hir().krate().modules.keys() {
362             tcx.ensure().check_mod_item_types(module);
363         }
364     });
365     tcx.sess.abort_if_errors();
366     tcx.sess.time("missing_docs", || {
367         rustc_lint::check_crate(tcx, rustc_lint::builtin::MissingDoc::new);
368     });
369     tcx.sess.time("check_mod_attrs", || {
370         for &module in tcx.hir().krate().modules.keys() {
371             tcx.ensure().check_mod_attrs(module);
372         }
373     });
374     rustc_passes::stability::check_unused_or_stable_features(tcx);
375
376     let access_levels = tcx.privacy_access_levels(());
377     // Convert from a HirId set to a DefId set since we don't always have easy access
378     // to the map from defid -> hirid
379     let access_levels = AccessLevels {
380         map: access_levels
381             .map
382             .iter()
383             .map(|(&k, &v)| (tcx.hir().local_def_id(k).to_def_id(), v))
384             .collect(),
385     };
386
387     let mut ctxt = DocContext {
388         tcx,
389         resolver,
390         param_env: ParamEnv::empty(),
391         external_traits: Default::default(),
392         active_extern_traits: Default::default(),
393         ty_substs: Default::default(),
394         lt_substs: Default::default(),
395         ct_substs: Default::default(),
396         impl_trait_bounds: Default::default(),
397         generated_synthetics: Default::default(),
398         auto_traits: tcx
399             .all_traits(())
400             .iter()
401             .cloned()
402             .filter(|trait_def_id| tcx.trait_is_auto(*trait_def_id))
403             .collect(),
404         module_trait_cache: FxHashMap::default(),
405         cache: Cache::new(access_levels, render_options.document_private),
406         inlined: FxHashSet::default(),
407         output_format,
408         render_options,
409     };
410
411     // Small hack to force the Sized trait to be present.
412     //
413     // Note that in case of `#![no_core]`, the trait is not available.
414     if let Some(sized_trait_did) = ctxt.tcx.lang_items().sized_trait() {
415         let mut sized_trait = build_external_trait(&mut ctxt, sized_trait_did);
416         sized_trait.is_auto = true;
417         ctxt.external_traits
418             .borrow_mut()
419             .insert(sized_trait_did, TraitWithExtraInfo { trait_: sized_trait, is_notable: false });
420     }
421
422     debug!("crate: {:?}", tcx.hir().krate());
423
424     let mut krate = tcx.sess.time("clean_crate", || clean::krate(&mut ctxt));
425
426     if krate.module.doc_value().map(|d| d.is_empty()).unwrap_or(true) {
427         let help = format!(
428             "The following guide may be of use:\n\
429             {}/rustdoc/how-to-write-documentation.html",
430             crate::DOC_RUST_LANG_ORG_CHANNEL
431         );
432         tcx.struct_lint_node(
433             crate::lint::MISSING_CRATE_LEVEL_DOCS,
434             DocContext::as_local_hir_id(tcx, krate.module.def_id).unwrap(),
435             |lint| {
436                 let mut diag =
437                     lint.build("no documentation found for this crate's top-level module");
438                 diag.help(&help);
439                 diag.emit();
440             },
441         );
442     }
443
444     fn report_deprecated_attr(name: &str, diag: &rustc_errors::Handler, sp: Span) {
445         let mut msg =
446             diag.struct_span_warn(sp, &format!("the `#![doc({})]` attribute is deprecated", name));
447         msg.note(
448             "see issue #44136 <https://github.com/rust-lang/rust/issues/44136> \
449              for more information",
450         );
451
452         if name == "no_default_passes" {
453             msg.help("you may want to use `#![doc(document_private_items)]`");
454         } else if name.starts_with("plugins") {
455             msg.warn("`#![doc(plugins = \"...\")]` no longer functions; see CVE-2018-1000622 <https://nvd.nist.gov/vuln/detail/CVE-2018-1000622>");
456         }
457
458         msg.emit();
459     }
460
461     let parse_pass = |name: &str, sp: Option<Span>| {
462         if let Some(pass) = passes::find_pass(name) {
463             Some(ConditionalPass::always(pass))
464         } else {
465             let msg = &format!("ignoring unknown pass `{}`", name);
466             let mut warning = if let Some(sp) = sp {
467                 tcx.sess.struct_span_warn(sp, msg)
468             } else {
469                 tcx.sess.struct_warn(msg)
470             };
471             if name == "collapse-docs" {
472                 warning.note("the `collapse-docs` pass was removed in #80261 <https://github.com/rust-lang/rust/pull/80261>");
473             }
474             warning.emit();
475             None
476         }
477     };
478
479     let mut manual_passes: Vec<_> =
480         manual_passes.into_iter().flat_map(|name| parse_pass(&name, None)).collect();
481
482     // Process all of the crate attributes, extracting plugin metadata along
483     // with the passes which we are supposed to run.
484     for attr in krate.module.attrs.lists(sym::doc) {
485         let diag = ctxt.sess().diagnostic();
486
487         let name = attr.name_or_empty();
488         if attr.is_word() {
489             if name == sym::no_default_passes {
490                 report_deprecated_attr("no_default_passes", diag, attr.span());
491                 if default_passes == passes::DefaultPassOption::Default {
492                     default_passes = passes::DefaultPassOption::None;
493                 }
494             }
495         } else if let Some(value) = attr.value_str() {
496             match name {
497                 sym::passes => {
498                     report_deprecated_attr("passes = \"...\"", diag, attr.span());
499                 }
500                 sym::plugins => {
501                     report_deprecated_attr("plugins = \"...\"", diag, attr.span());
502                     continue;
503                 }
504                 _ => continue,
505             };
506             for name in value.as_str().split_whitespace() {
507                 let span = attr.name_value_literal_span().unwrap_or(attr.span());
508                 manual_passes.extend(parse_pass(name, Some(span)));
509             }
510         }
511
512         if attr.is_word() && name == sym::document_private_items {
513             ctxt.render_options.document_private = true;
514         }
515     }
516
517     let passes = passes::defaults(default_passes).iter().copied().chain(manual_passes);
518     info!("Executing passes");
519
520     for p in passes {
521         let run = match p.condition {
522             Always => true,
523             WhenDocumentPrivate => ctxt.render_options.document_private,
524             WhenNotDocumentPrivate => !ctxt.render_options.document_private,
525             WhenNotDocumentHidden => !ctxt.render_options.document_hidden,
526         };
527         if run {
528             debug!("running pass {}", p.pass.name);
529             krate = ctxt.tcx.sess.time(p.pass.name, || (p.pass.run)(krate, &mut ctxt));
530         }
531     }
532
533     ctxt.sess().abort_if_errors();
534
535     let render_options = ctxt.render_options;
536     let mut cache = ctxt.cache;
537     krate = tcx.sess.time("create_format_cache", || {
538         cache.populate(krate, tcx, &render_options.extern_html_root_urls, &render_options.output)
539     });
540
541     // The main crate doc comments are always collapsed.
542     krate.collapsed = true;
543
544     (krate, render_options, cache)
545 }
546
547 /// Due to <https://github.com/rust-lang/rust/pull/73566>,
548 /// the name resolution pass may find errors that are never emitted.
549 /// If typeck is called after this happens, then we'll get an ICE:
550 /// 'Res::Error found but not reported'. To avoid this, emit the errors now.
551 struct EmitIgnoredResolutionErrors<'tcx> {
552     tcx: TyCtxt<'tcx>,
553 }
554
555 impl<'tcx> EmitIgnoredResolutionErrors<'tcx> {
556     fn new(tcx: TyCtxt<'tcx>) -> Self {
557         Self { tcx }
558     }
559 }
560
561 impl<'tcx> Visitor<'tcx> for EmitIgnoredResolutionErrors<'tcx> {
562     type Map = Map<'tcx>;
563
564     fn nested_visit_map(&mut self) -> NestedVisitorMap<Self::Map> {
565         // We need to recurse into nested closures,
566         // since those will fallback to the parent for type checking.
567         NestedVisitorMap::OnlyBodies(self.tcx.hir())
568     }
569
570     fn visit_path(&mut self, path: &'tcx Path<'_>, _id: HirId) {
571         debug!("visiting path {:?}", path);
572         if path.res == Res::Err {
573             // We have less context here than in rustc_resolve,
574             // so we can only emit the name and span.
575             // However we can give a hint that rustc_resolve will have more info.
576             let label = format!(
577                 "could not resolve path `{}`",
578                 path.segments
579                     .iter()
580                     .map(|segment| segment.ident.as_str().to_string())
581                     .collect::<Vec<_>>()
582                     .join("::")
583             );
584             let mut err = rustc_errors::struct_span_err!(
585                 self.tcx.sess,
586                 path.span,
587                 E0433,
588                 "failed to resolve: {}",
589                 label
590             );
591             err.span_label(path.span, label);
592             err.note("this error was originally ignored because you are running `rustdoc`");
593             err.note("try running again with `rustc` or `cargo check` and you may get a more detailed error");
594             err.emit();
595         }
596         // We could have an outer resolution that succeeded,
597         // but with generic parameters that failed.
598         // Recurse into the segments so we catch those too.
599         intravisit::walk_path(self, path);
600     }
601 }
602
603 /// `DefId` or parameter index (`ty::ParamTy.index`) of a synthetic type parameter
604 /// for `impl Trait` in argument position.
605 #[derive(Clone, Copy, PartialEq, Eq, Hash, PartialOrd, Ord)]
606 crate enum ImplTraitParam {
607     DefId(DefId),
608     ParamIndex(u32),
609 }
610
611 impl From<DefId> for ImplTraitParam {
612     fn from(did: DefId) -> Self {
613         ImplTraitParam::DefId(did)
614     }
615 }
616
617 impl From<u32> for ImplTraitParam {
618     fn from(idx: u32) -> Self {
619         ImplTraitParam::ParamIndex(idx)
620     }
621 }