diff options
| author | Björn Gustavsson <[email protected]> | 2012-11-16 07:26:07 +0100 |
|---|---|---|
| committer | Björn Gustavsson <[email protected]> | 2012-12-06 14:22:36 +0100 |
| commit | 81d6790acf9d55a63575efd93bdba1629c101b62 (patch) | |
| tree | 319b38eda076912823460c2e4302d7e9f9756e93 /lib/asn1/src/asn1ct_constructed_per.erl | |
| parent | ca3b66e3d2c734e8715c7ed078e4fa9ae10462de (diff) | |
| download | otp-81d6790acf9d55a63575efd93bdba1629c101b62.tar.gz otp-81d6790acf9d55a63575efd93bdba1629c101b62.tar.bz2 otp-81d6790acf9d55a63575efd93bdba1629c101b62.zip | |
Suppress warnings for unused variables using a compiler option
Currently, the generated code suppresses warnings for unusued variables
by assigning to the "_Val" variable, for example:
_Val = Tmpval5
To be completely safe, that should have been:
_ = Tmpval5
in case there happens to more than one such assignment in the same
scope.
However, a better way to suppress warnings is to simply use the
'nowarn_unused_vars' compiler option. That method will also work
for more complicated expressions there it would be cumbersome to
suppress warnings by assignment to "_".
Diffstat (limited to 'lib/asn1/src/asn1ct_constructed_per.erl')
| -rw-r--r-- | lib/asn1/src/asn1ct_constructed_per.erl | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/lib/asn1/src/asn1ct_constructed_per.erl b/lib/asn1/src/asn1ct_constructed_per.erl index e4e0e064e8..46034622c8 100644 --- a/lib/asn1/src/asn1ct_constructed_per.erl +++ b/lib/asn1/src/asn1ct_constructed_per.erl @@ -76,9 +76,7 @@ gen_encode_constructed(Erule,Typename,D) when is_record(D,type) -> case {Optionals = optionals(to_textual_order(CompList)),CompList, is_optimized(Erule)} of {[],EmptyCL,_} when EmptyCL == {[],[],[]};EmptyCL == {[],[]};EmptyCL == [] -> - emit(["%%Variable setting just to eliminate ", - "compiler warning for unused vars!",nl, - "_Val = ",{curr,val},",",nl]); + ok; {[],_,_} -> emit([{next,val}," = ",{curr,val},",",nl]); {_,_,true} -> |