@@ -432,14 +432,12 @@ fn build_source_change(
432432 assoc_item_list. add_items ( & mut editor, items. clone ( ) ) ;
433433
434434 // Add tabstop for last method
435- if let Some ( cap) = ctx. config . snippet_cap {
436- if let Some ( ast:: AssocItem :: Fn ( fn_) ) = items. last ( ) {
437- if let Some ( name) = fn_. name ( ) {
435+ if let Some ( cap) = ctx. config . snippet_cap
436+ && let Some ( ast:: AssocItem :: Fn ( fn_) ) = items. last ( )
437+ && let Some ( name) = fn_. name ( ) {
438438 let tabstop = builder. make_tabstop_before ( cap) ;
439439 editor. add_annotation ( name. syntax ( ) . clone ( ) , tabstop) ;
440440 }
441- }
442- }
443441
444442 builder. add_file_edits ( ctx. vfs_file_id ( ) , editor) ;
445443 } else {
@@ -463,16 +461,13 @@ fn build_source_change(
463461 ) ;
464462
465463 // Add tabstop for last method
466- if let Some ( cap) = ctx. config . snippet_cap {
467- if let Some ( assoc_list) = impl_def. assoc_item_list ( ) {
468- if let Some ( ast:: AssocItem :: Fn ( fn_) ) = assoc_list. assoc_items ( ) . last ( ) {
469- if let Some ( name) = fn_. name ( ) {
464+ if let Some ( cap) = ctx. config . snippet_cap
465+ && let Some ( assoc_list) = impl_def. assoc_item_list ( )
466+ && let Some ( ast:: AssocItem :: Fn ( fn_) ) = assoc_list. assoc_items ( ) . last ( )
467+ && let Some ( name) = fn_. name ( ) {
470468 let tabstop = builder. make_tabstop_before ( cap) ;
471469 editor. add_annotation ( name. syntax ( ) . clone ( ) , tabstop) ;
472470 }
473- }
474- }
475- }
476471
477472 builder. add_file_edits ( ctx. vfs_file_id ( ) , editor) ;
478473 }
0 commit comments