Skip to content

Commit

Permalink
sockaddr: Turn FIXME comment into XXX
Browse files Browse the repository at this point in the history
The LGTM service constantly annoys me with this alert, which in fact
isn't actually a thing that's broken, because the function in question
is just slower than it could be.

This is going to be addressed at some point[1] but it's not something
critical and since "FIXME" is the wrong label anyway, let's get rid of
that annoying alert.

[1]: #25

Signed-off-by: aszlig <[email protected]>
  • Loading branch information
aszlig committed Jul 7, 2021
1 parent de8ea92 commit 4e4a727
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/sockaddr.cc
Original file line number Diff line number Diff line change
Expand Up @@ -244,7 +244,7 @@ bool SockAddr::operator==(const SockAddr &other) const

std::size_t SockAddr::get_hash(void) const
{
// FIXME: This function is pretty slow and dumb, but at least accurate.
// XXX: This function is pretty slow and dumb, but at least accurate.
std::ostringstream hashprep;
hashprep << this->ss_family;

Expand Down

0 comments on commit 4e4a727

Please sign in to comment.