all: fix some typos in comments and names

[georgehao]

This pull request includes several changes to improve code clarity and consistency across multiple files. The changes mainly involve adding detailed comments to functions and renaming variables and functions for better readability.

Improvements to Comments and Documentation:

• accounts/accounts.go: Added detailed comments to WalletDropped to explain when the event is fired.
• cmd/devp2p/internal/v4test/discv4tests.go: Updated comments for multiple functions to provide clear descriptions of their behavior, such as PingExtraDataWrongFrom, PingPastExpiration, WrongPacketType, and others. [1] [2] [3] [4] [5] [6] [7] [8] [9]
• core/types/deposit.go: Renamed UnpackIntoDeposit to DepositLogToRequest and updated the comment for clarity.

Variable and Function Renaming:

• core/txpool/legacypool/legacypool2_test.go: Renamed cur_balance to curBalance for consistency.
• core/vm/eof_validation.go: Renamed max_size to maxSize for consistency. [1] [2]
• crypto/ecies/ecies.go: Renamed several functions for clarity, such as Export to ExportECDSA, Import to ImportECDSAPublic, and Generate to GenerateKey. [1] [2] [3]

Code Formatting:

• cmd/devp2p/internal/ethtest/protocol.go: Added a newline for better readability.
• core/vm/program/program.go: Added a newline for better readability. [1] [2] [3]
• crypto/blake2b/blake2b.go: Updated comments to follow a consistent format.
• crypto/bn256/gnark/pairing.go: Updated comments for clarity.
• eth/tracers/dir.go: Updated comments for clarity.

Miscellaneous:

• consensus/clique/snapshot_test.go: Removed an unnecessary break statement.
• core/tracing/hooks.go: Updated comment for GasChangeCallContractCreation2 for clarity.
• metrics/gauge_info.go: Updated comment for GaugeInfoSnapshot for clarity.

These changes collectively enhance the readability and maintainability of the codebase.

[holiman]

Thanks, I think these are worthwhile changes. LGTM!