redundant pragma deprecated warning at the site of declaration

[YashasSamaga]

#pragma deprecated don't use please
f() {

}

main() {
  
}

no deprecated function warning for the above code which is right

#pragma deprecated don't use please
f() {

}

main() {
  f();
}

two warnings are thrown instead of just one

I think this is not right. : the warning thrown at the site of declaration of the deprecated function is redundant.

[YashasSamaga]

The fix is as simple as deleting one line: error(234,symbolname,ptr); /* deprecated (probably a public function) */

It appears like it has been intentionally kept to trigger warnings at the site of declaration. There might be a reason why it's there.

[Southclaws]

I'm down for removing this - it makes little sense to trigger a warning at the decl because the person seeing this error is most likely a library consumer, not the author so they are more concerned about their own code rather than the library that provides that API.

[Zeex]

I think this is supposed to handle cases when a forward declaration is marked as deprecated and somebody declares a (public?) function with the same name/prototype.

#include <a_samp>

#pragma deprecated
forward hello();

public hello() {
	print("I'm deprecated!");
}

main() {
}

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[Y-Less]

Seems this isn't solved, I thought it was.