Skip to content

Commit

Permalink
Resolve Flutter 3.29 analyze issues
Browse files Browse the repository at this point in the history
  • Loading branch information
JSUYA committed Feb 18, 2025
1 parent c6cb477 commit 3ad001a
Show file tree
Hide file tree
Showing 4 changed files with 4 additions and 10 deletions.
2 changes: 0 additions & 2 deletions analysis_options.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -127,7 +127,6 @@ linter:
# - one_member_abstracts # too many false positives
- only_throw_errors # this does get disabled in a few places where we have legacy code that uses strings et al
- overridden_fields
- package_api_docs
- package_names
- package_prefixed_library_names
# - parameter_assignments # we do this commonly
Expand Down Expand Up @@ -215,7 +214,6 @@ linter:
- unnecessary_to_list_in_spreads
- unreachable_from_main
- unrelated_type_equality_checks
- unsafe_html
- use_build_context_synchronously
- use_colored_box
# - use_decorated_box # leads to bugs: DecoratedBox and Container are not equivalent (Container inserts extra padding)
Expand Down
Original file line number Diff line number Diff line change
@@ -1,12 +1,11 @@
// ignore: avoid_web_libraries_in_flutter
import 'dart:html' as html;
import 'package:web/web.dart' as web;

class DeviceInfo {
static String get label {
return 'Flutter Web';
}

static String get userAgent {
return 'flutter-webrtc/web-plugin 0.0.1 ( ${html.window.navigator.userAgent} )';
return 'flutter-webrtc/web-plugin 0.0.1 ( ${web.window.navigator.userAgent} )';
}
}
Original file line number Diff line number Diff line change
@@ -1,10 +1,10 @@
// ignore: uri_does_not_exist
import 'dart:core';
import 'dart:html' as html;

import 'package:flutter/foundation.dart';
import 'package:flutter/material.dart';
import 'package:flutter_webrtc/flutter_webrtc.dart';
import 'package:web/web.dart' as web;

/*
* getUserMedia sample
Expand Down Expand Up @@ -114,8 +114,7 @@ class _GetUserMediaSampleState extends State<GetUserMediaSample> {
_mediaRecorder = null;
});
print(objectUrl);
// ignore: unsafe_html
html.window.open(objectUrl, '_blank');
web.window.open(objectUrl, '_blank');
}

void _captureFrame() async {
Expand Down
2 changes: 0 additions & 2 deletions packages/sqflite/example/analysis_options.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,6 @@ linter:
- no_duplicate_case_values
- non_constant_identifier_names
- omit_local_variable_types
- package_api_docs
- package_prefixed_library_names
- prefer_generic_function_type_aliases
- prefer_is_empty
Expand All @@ -63,7 +62,6 @@ linter:
- unnecessary_null_aware_assignments
- unnecessary_statements
- unrelated_type_equality_checks
- unsafe_html
- valid_regexps

- constant_identifier_names
Expand Down

0 comments on commit 3ad001a

Please sign in to comment.