Grep for space indentation (#54850)

#54604 atomizing
Since a lot of the space indents are in lists ill atomize those later

code/modules/instruments/instrument_data/_instrument_data.dm

@@ -1,6 +1,6 @@
 /**
-  * Get all non admin_only instruments as a list of text ids.
-  */
+ * Get all non admin_only instruments as a list of text ids.
+ */
 /proc/get_allowed_instrument_ids()
 	. = list()
 	for(var/id in SSinstruments.instrument_data)
@@ -9,13 +9,13 @@
 			. += I.id
 
 /**
-  * # Instrument Datums
-  *
-  * Instrument datums hold the data for any given instrument, as well as data on how to play it and what bounds there are to playing it.
-  *
-  * The datums themselves are kept in SSinstruments in a list by their unique ID. The reason it uses ID instead of typepath is to support the runtime creation of instruments.
-  * Since songs cache them while playing, there isn't realistic issues regarding performance from accessing.
-  */
+ * # Instrument Datums
+ *
+ * Instrument datums hold the data for any given instrument, as well as data on how to play it and what bounds there are to playing it.
+ *
+ * The datums themselves are kept in SSinstruments in a list by their unique ID. The reason it uses ID instead of typepath is to support the runtime creation of instruments.
+ * Since songs cache them while playing, there isn't realistic issues regarding performance from accessing.
+ */
 /datum/instrument
 	/// Name of the instrument
 	var/name = "Generic instrument"
@@ -51,16 +51,16 @@
 		id = "[type]"
 
 /**
-  * Initializes the instrument, calculating its samples if necessary.
-  */
+ * Initializes the instrument, calculating its samples if necessary.
+ */
 /datum/instrument/proc/Initialize()
 	if(instrument_flags & (INSTRUMENT_LEGACY | INSTRUMENT_DO_NOT_AUTOSAMPLE))
 		return
 	calculate_samples()
 
 /**
-  * Checks if this instrument is ready to play.
-  */
+ * Checks if this instrument is ready to play.
+ */
 /datum/instrument/proc/ready()
 	if(instrument_flags & INSTRUMENT_LEGACY)
 		return legacy_instrument_path && legacy_instrument_ext
@@ -79,9 +79,9 @@
 	return ..()
 
 /**
-  * For synthesized instruments, this is how the instrument generates the "keys" that a [/datum/song] uses to play notes.
-  * Calculating them on the fly would be unperformant, so we do it during init and keep it all cached in a list.
-  */
+ * For synthesized instruments, this is how the instrument generates the "keys" that a [/datum/song] uses to play notes.
+ * Calculating them on the fly would be unperformant, so we do it during init and keep it all cached in a list.
+ */
 /datum/instrument/proc/calculate_samples()
 	if(!length(real_samples))
 		CRASH("No real samples defined for [id] [type] on calculate_samples() call.")

code/modules/instruments/instrument_data/_instrument_key.dm

@@ -1,7 +1,7 @@
 /**
-  * Instrument key datums contain everything needed to know how to play a specific
-  * note of an instrument.*
-  */
+ * Instrument key datums contain everything needed to know how to play a specific
+ * note of an instrument.*
+ */
 /datum/instrument_key
 	/// The numerical key of what this is, from 1 to 127 on a standard piano keyboard.
 	var/key
@@ -21,8 +21,8 @@
 		calculate()
 
 /**
-  * Calculates and stores our deviation.
-  */
+ * Calculates and stores our deviation.
+ */
 /datum/instrument_key/proc/calculate()
 	if(!deviation)
 		CRASH("Invalid calculate call: No deviation or sample in instrument_key")

code/modules/instruments/items.dm

@@ -93,15 +93,15 @@
 	RegisterSignal(src, COMSIG_SONG_END, .proc/stop_playing)
 
 /**
-  * Called by a component signal when our song starts playing.
-  */
+ * Called by a component signal when our song starts playing.
+ */
 /obj/item/instrument/piano_synth/headphones/proc/start_playing()
 	icon_state = "[initial(icon_state)]_on"
 	update_icon()
 
 /**
-  * Called by a component signal when our song stops playing.
-  */
+ * Called by a component signal when our song stops playing.
+ */
 /obj/item/instrument/piano_synth/headphones/proc/stop_playing()
 	icon_state = "[initial(icon_state)]"
 	update_icon()

code/modules/instruments/songs/_song.dm

@@ -3,11 +3,11 @@
 #define MUSIC_MAXLINECHARS 300
 
 /**
-  * # Song datum
-  *
-  * These are the actual backend behind instruments.
-  * They attach to an atom and provide the editor + playback functionality.
-  */
+ * # Song datum
+ *
+ * These are the actual backend behind instruments.
+ * They attach to an atom and provide the editor + playback functionality.
+ */
 /datum/song
 	/// Name of the song
 	var/name = "Untitled"
@@ -152,8 +152,8 @@
 	return ..()
 
 /**
-  * Checks and stores which mobs can hear us. Terminates sounds for mobs that leave our range.
-  */
+ * Checks and stores which mobs can hear us. Terminates sounds for mobs that leave our range.
+ */
 /datum/song/proc/do_hearcheck()
 	last_hearcheck = world.time
 	var/list/old = hearing_mobs.Copy()
@@ -166,8 +166,8 @@
 		terminate_sound_mob(i)
 
 /**
-  * Sets our instrument, caching anything necessary for faster accessing. Accepts an ID, typepath, or instantiated instrument datum.
-  */
+ * Sets our instrument, caching anything necessary for faster accessing. Accepts an ID, typepath, or instantiated instrument datum.
+ */
 /datum/song/proc/set_instrument(datum/instrument/I)
 	terminate_all_sounds()
 	var/old_legacy
@@ -197,8 +197,8 @@
 				compile_chords()
 
 /**
-  * Attempts to start playing our song.
-  */
+ * Attempts to start playing our song.
+ */
 /datum/song/proc/start_playing(mob/user)
 	if(playing)
 		return
@@ -222,8 +222,8 @@
 	START_PROCESSING(SSinstruments, src)
 
 /**
-  * Stops playing, terminating all sounds if in synthesized mode. Clears hearing_mobs.
-  */
+ * Stops playing, terminating all sounds if in synthesized mode. Clears hearing_mobs.
+ */
 /datum/song/proc/stop_playing()
 	if(!playing)
 		return
@@ -237,8 +237,8 @@
 	user_playing = null
 
 /**
-  * Processes our song.
-  */
+ * Processes our song.
+ */
 /datum/song/proc/process_song(wait)
 	if(!length(compiled_chords) || should_stop_playing(user_playing))
 		stop_playing()
@@ -259,55 +259,55 @@
 				return
 
 /**
-  * Converts a tempodiv to ticks to elapse before playing the next chord, taking into account our tempo.
-  */
+ * Converts a tempodiv to ticks to elapse before playing the next chord, taking into account our tempo.
+ */
 /datum/song/proc/tempodiv_to_delay(tempodiv)
 	if(!tempodiv)
 		tempodiv = 1		// no division by 0. some song converters tend to use 0 for when it wants to have no div, for whatever reason.
 	return max(1, round((tempo/tempodiv) / world.tick_lag, 1))
 
 /**
-  * Compiles chords.
-  */
+ * Compiles chords.
+ */
 /datum/song/proc/compile_chords()
 	legacy? compile_legacy() : compile_synthesized()
 
 /**
-  * Plays a chord.
-  */
+ * Plays a chord.
+ */
 /datum/song/proc/play_chord(list/chord)
 	// last value is timing information
 	for(var/i in 1 to (length(chord) - 1))
 		legacy? playkey_legacy(chord[i][1], chord[i][2], chord[i][3], user_playing) : playkey_synth(chord[i], user_playing)
 
 /**
-  * Checks if we should halt playback.
-  */
+ * Checks if we should halt playback.
+ */
 /datum/song/proc/should_stop_playing(mob/user)
 	return QDELETED(parent) || !using_instrument || !playing
 
 /**
-  * Sanitizes tempo to a value that makes sense and fits the current world.tick_lag.
-  */
+ * Sanitizes tempo to a value that makes sense and fits the current world.tick_lag.
+ */
 /datum/song/proc/sanitize_tempo(new_tempo)
 	new_tempo = abs(new_tempo)
 	return clamp(round(new_tempo, world.tick_lag), world.tick_lag, 5 SECONDS)
 
 /**
-  * Gets our beats per minute based on our tempo.
-  */
+ * Gets our beats per minute based on our tempo.
+ */
 /datum/song/proc/get_bpm()
 	return 600 / tempo
 
 /**
-  * Sets our tempo from a beats-per-minute, sanitizing it to a valid number first.
-  */
+ * Sets our tempo from a beats-per-minute, sanitizing it to a valid number first.
+ */
 /datum/song/proc/set_bpm(bpm)
 	tempo = sanitize_tempo(600 / bpm)
 
 /**
-  * Updates the window for our users. Override down the line.
-  */
+ * Updates the window for our users. Override down the line.
+ */
 /datum/song/proc/updateDialog(mob/user)
 	ui_interact(user)
 
@@ -319,8 +319,8 @@
 	process_decay(world.tick_lag)
 
 /**
-  * Updates our cached linear/exponential falloff stuff, saving calculations down the line.
-  */
+ * Updates our cached linear/exponential falloff stuff, saving calculations down the line.
+ */
 /datum/song/proc/update_sustain()
 	// Exponential is easy
 	cached_exponential_dropoff = sustain_exponential_dropoff
@@ -331,32 +331,32 @@
 	cached_linear_dropoff = volume_decrease_per_decisecond
 
 /**
-  * Setter for setting output volume.
-  */
+ * Setter for setting output volume.
+ */
 /datum/song/proc/set_volume(volume)
 	src.volume = clamp(volume, max(0, min_volume), min(100, max_volume))
 	update_sustain()
 	updateDialog()
 
 /**
-  * Setter for setting how low the volume has to get before a note is considered "dead" and dropped
-  */
+ * Setter for setting how low the volume has to get before a note is considered "dead" and dropped
+ */
 /datum/song/proc/set_dropoff_volume(volume)
 	sustain_dropoff_volume = clamp(volume, INSTRUMENT_MIN_SUSTAIN_DROPOFF, 100)
 	update_sustain()
 	updateDialog()
 
 /**
-  * Setter for setting exponential falloff factor.
-  */
+ * Setter for setting exponential falloff factor.
+ */
 /datum/song/proc/set_exponential_drop_rate(drop)
 	sustain_exponential_dropoff = clamp(drop, INSTRUMENT_EXP_FALLOFF_MIN, INSTRUMENT_EXP_FALLOFF_MAX)
 	update_sustain()
 	updateDialog()
 
 /**
-  * Setter for setting linear falloff duration.
-  */
+ * Setter for setting linear falloff duration.
+ */
 /datum/song/proc/set_linear_falloff_duration(duration)
 	sustain_linear_duration = clamp(duration, 0.1, INSTRUMENT_MAX_TOTAL_SUSTAIN)
 	update_sustain()

code/modules/instruments/songs/editor.dm

@@ -1,6 +1,6 @@
 /**
-  * Returns the HTML for the status UI for this song datum.
-  */
+ * Returns the HTML for the status UI for this song datum.
+ */
 /datum/song/proc/instrument_status_ui()
 	. = list()
 	. += "<div class='statusDisplay'>"
@@ -88,8 +88,8 @@
 	popup.open()
 
 /**
-  * Parses a song the user has input into lines and stores them.
-  */
+ * Parses a song the user has input into lines and stores them.
+ */
 /datum/song/proc/ParseSong(text)
 	set waitfor = FALSE
 	//split into lines

code/modules/instruments/songs/play_legacy.dm

@@ -1,6 +1,6 @@
 /**
-  * Compiles our lines into "chords" with filenames for legacy playback. This makes there have to be a bit of lag at the beginning of the song, but repeats will not have to parse it again, and overall playback won't be impacted by as much lag.
-  */
+ * Compiles our lines into "chords" with filenames for legacy playback. This makes there have to be a bit of lag at the beginning of the song, but repeats will not have to parse it again, and overall playback won't be impacted by as much lag.
+ */
 /datum/song/proc/compile_legacy()
 	if(!length(src.lines))
 		return
@@ -39,13 +39,13 @@
 				compiled_chords[++compiled_chords.len] = compiled_chord
 
 /**
-  * Proc to play a legacy note. Just plays the sound to hearing mobs (and does hearcheck if necessary), no fancy channel/sustain/management.
-  *
-  * Arguments:
-  * * note is a number from 1-7 for A-G
-  * * acc is either "b", "n", or "#"
-  * * oct is 1-8 (or 9 for C)
-  */
+ * Proc to play a legacy note. Just plays the sound to hearing mobs (and does hearcheck if necessary), no fancy channel/sustain/management.
+ *
+ * Arguments:
+ * * note is a number from 1-7 for A-G
+ * * acc is either "b", "n", or "#"
+ * * oct is 1-8 (or 9 for C)
+ */
 /datum/song/proc/playkey_legacy(note, acc as text, oct, mob/user)
 	// handle accidental -> B<>C of E<>F
 	if(acc == "b" && (note == 3 || note == 6)) // C or F

code/modules/instruments/songs/play_synthesized.dm

@@ -1,6 +1,6 @@
 /**
-  * Compiles our lines into "chords" with numbers. This makes there have to be a bit of lag at the beginning of the song, but repeats will not have to parse it again, and overall playback won't be impacted by as much lag.
-  */
+ * Compiles our lines into "chords" with numbers. This makes there have to be a bit of lag at the beginning of the song, but repeats will not have to parse it again, and overall playback won't be impacted by as much lag.
+ */
 /datum/song/proc/compile_synthesized()
 	if(!length(src.lines))
 		return
@@ -39,9 +39,9 @@
 				compiled_chords[++compiled_chords.len] = compiled_chord
 
 /**
-  * Plays a specific numerical key from our instrument to anyone who can hear us.
-  * Does a hearing check if enough time has passed.
-  */
+ * Plays a specific numerical key from our instrument to anyone who can hear us.
+ * Does a hearing check if enough time has passed.
+ */
 /datum/song/proc/playkey_synth(key, mob/user)
 	if(can_noteshift)
 		key = clamp(key + note_shift, key_min, key_max)
@@ -71,8 +71,8 @@
 		// Could do environment and echo later but not for now
 
 /**
-  * Stops all sounds we are "responsible" for. Only works in synthesized mode.
-  */
+ * Stops all sounds we are "responsible" for. Only works in synthesized mode.
+ */
 /datum/song/proc/terminate_all_sounds(clear_channels = TRUE)
 	for(var/i in hearing_mobs)
 		terminate_sound_mob(i)
@@ -84,15 +84,15 @@
 		SSsounds.free_datum_channels(src)
 
 /**
-  * Stops all sounds we are responsible for in a given person. Only works in synthesized mode.
-  */
+ * Stops all sounds we are responsible for in a given person. Only works in synthesized mode.
+ */
 /datum/song/proc/terminate_sound_mob(mob/M)
 	for(var/channel in channels_playing)
 		M.stop_sound_channel(text2num(channel))
 
 /**
-  * Pops a channel we have reserved so we don't have to release and re-request them from SSsounds every time we play a note. This is faster.
-  */
+ * Pops a channel we have reserved so we don't have to release and re-request them from SSsounds every time we play a note. This is faster.
+ */
 /datum/song/proc/pop_channel()
 	if(length(channels_idle))			//just pop one off of here if we have one available
 		. = text2num(channels_idle[1])
@@ -105,11 +105,11 @@
 		using_sound_channels++
 
 /**
-  * Decays our channels and updates their volumes to mobs who can hear us.
-  *
-  * Arguments:
-  * * wait_ds - the deciseconds we should decay by. This is to compensate for any lag, as otherwise songs would get pretty nasty during high time dilation.
-  */
+ * Decays our channels and updates their volumes to mobs who can hear us.
+ *
+ * Arguments:
+ * * wait_ds - the deciseconds we should decay by. This is to compensate for any lag, as otherwise songs would get pretty nasty during high time dilation.
+ */
 /datum/song/proc/process_decay(wait_ds)
 	var/linear_dropoff = cached_linear_dropoff * wait_ds
 	var/exponential_dropoff = cached_exponential_dropoff ** wait_ds