 #[zstack Test Bot Report] P A S S
 Integration Test Result:
Test Summary:

 TestSuite|TestCase | Pass | Fail | Timeout | Skip
 ---------|---------| ---- | ---- | ------- | ----
 basic_test|__suite_setup__|0|1|0|0
basic_test|_basic/test_create_vm_|0|0|0|1
basic_test|_basic/test_stop_vm_|0|0|0|1
basic_test|_basic/test_start_vm_|0|0|0|1
basic_test|_basic/test_reboot_vm_|0|0|0|1
basic_test|_basic/test_add_volume_|0|0|0|1
basic_test|_basic/test_add_vol_to_stopvm_|0|0|0|1
basic_test|_basic/test_add_volume_reboot_vm_|0|0|0|1
basic_test|_basic/test_add_multi_volumes_|0|0|0|1
basic_test|_basic/test_add_volume_negative_|0|0|0|1
basic_test|_basic/test_crt_temp_from_volume_|0|0|0|1
basic_test|_basic/test_crt_vm_with_volume_|0|0|0|1
basic_test|_basic/test_vm_securitygroup_|0|0|0|1
basic_test|_basic/test_create_iso_vm_|0|0|0|1
basic_test|_account/test_crt_vm_by_normal_account_|0|0|0|1
basic_test|_account/test_crt_vm_by_normal_account2_|0|0|0|1
basic_test|_account/test_vm_ops_by_normal_account_|0|0|0|1
basic_test|_account/test_multi_accounts_vm_ops_|0|0|0|1
basic_test|_suite_teardown_|0|0|0|1
virtual_router_test|__suite_setup__|0|1|0|0
virtual_router_test|_host/test_reconnect_host_|0|0|0|1
virtual_router_test|_account/test_vm_ops_by_normal_account_|0|0|0|1
virtual_router_test|_account/test_multi_accounts_vm_ops_|0|0|0|1
virtual_router_test|_novlan/test_create_vm_|0|0|0|1
virtual_router_test|_novlan/test_reboot_vm_|0|0|0|1
virtual_router_test|_novlan/test_ping_vms_|0|0|0|1
virtual_router_test|_host/test_reconnect_host_|0|0|0|1
virtual_router_test|_vlan/test_create_vm_|0|0|0|1
virtual_router_test|_vlan/test_reboot_vm_|0|0|0|1
virtual_router_test|_vlan/test_snat_create_|0|0|0|1
virtual_router_test|_vlan/test_dnat_reboot_|0|0|0|1
virtual_router_test|_vlan/test_ping_vms_|0|0|0|1
virtual_router_test|_vlan/test_create_3vms_|0|0|0|1
virtual_router_test|_vlan/test_start_vm_|0|0|0|1
virtual_router_test|_vlan/test_3l3s_vm_|0|0|0|1
virtual_router_test|_vlan/test_remove_dhcp_|0|0|0|1
virtual_router_test|_vlan/test_2l3_vm_|0|0|0|1
virtual_router_test|_vlan/test_crt_vms_with_limited_ips_|0|0|0|1
virtual_router_test|_vlan/test_crt_vms_with_limited_ips2.0_|0|0|0|1
virtual_router_test|_vlan/test_crt_vms_with_limited_ips2.1_|0|0|0|1
virtual_router_test|_vlan/test_start_vms_simultaneously_|0|0|0|1
virtual_router_test|_vlan/test_crt_static_ip_vm_|0|0|0|1
virtual_router_test|_vlan/test_crt_vm_with_hostname_|0|0|0|1
virtual_router_test|_vlan/test_attach_nic_|0|0|0|1
virtual_router_test|_vlan/test_attach_nic_stopped_vm_|0|0|0|1
virtual_router_test|_vlan/test_attach_nic_reboot_vm_|0|0|0|1
virtual_router_test|_vlan/test_chg_static_ip_vm_|0|0|0|1
virtual_router_test|_vlan/test_attach_nic_with_static_ip_|0|0|0|1
virtual_router_test|_volume/test_add_volume_|0|0|0|1
virtual_router_test|_volume/test_add_m_vols_|0|0|0|1
virtual_router_test|_volume/test_add_volume_vr_|0|0|0|1
virtual_router_test|_volume/test_crt_vm_volume_|0|0|0|1
virtual_router_test|_volume/test_crt_vm_2_volumes_|0|0|0|1
virtual_router_test|_volume/test_crt_vm_24_volumes_|0|0|0|1
virtual_router_test|_volume/test_volume_ops_on_stopped_vm_|0|0|0|1
virtual_router_test|_volume/test_crt_vm_from_dvt_|0|0|0|1
virtual_router_test|_template_image/test_crt_temp_image_|0|0|0|1
virtual_router_test|_template_image/test_del_temp_image_|0|0|0|1
virtual_router_test|_sg/test_ingress_|0|0|0|1
virtual_router_test|_sg/test_egress_|0|0|0|1
virtual_router_test|_sg/test_icmp_|0|0|0|1
virtual_router_test|_sg/test_udp_|0|0|0|1
virtual_router_test|_sg/test_stopped_vm_|0|0|0|1
virtual_router_test|_sg/test_vm_ops_|0|0|0|1
virtual_router_test|_sg/test_2vm_egress_|0|0|0|1
virtual_router_test|_sg/test_2vm_ingress_|0|0|0|1
virtual_router_test|_sg/test_4vm_complx_|0|0|0|1
virtual_router_test|_sg/test_detach_l3_|0|0|0|1
virtual_router_test|_pf/test_2l3s_pfs_|0|0|0|1
virtual_router_test|_pf/test_attach_pf_|0|0|0|1
virtual_router_test|_pf/test_delete_pf_after_destroy_vm_|0|0|0|1
virtual_router_test|_pf/test_del_vip_before_add_pf_to_vm_|0|0|0|1
virtual_router_test|_pf/test_multi_pfs_sg_tcp_|0|0|0|1
virtual_router_test|_pf/test_multi_rules_|0|0|0|1
virtual_router_test|_pf/test_pf_after_destroy_vm_|0|0|0|1
virtual_router_test|_pf/test_pf_sg_tcp_|0|0|0|1
virtual_router_test|_pf/test_pfs_tcp_udp_same_vip_ports_|0|0|0|1
virtual_router_test|_pf/test_pfs_tcp_with_same_vip_|0|0|0|1
virtual_router_test|_pf/test_tcp_|0|0|0|1
virtual_router_test|_pf/test_udp_|0|0|0|1
virtual_router_test|_pf/test_vm_ops_|0|0|0|1
virtual_router_test|_eip/test_2_eips_|0|0|0|1
virtual_router_test|_eip/test_2l3s_eip_pf_|0|0|0|1
virtual_router_test|_eip/test_2l3s_eips_|0|0|0|1
virtual_router_test|_eip/test_attach_eip_|0|0|0|1
virtual_router_test|_eip/test_create_eip_vm_|0|0|0|1
virtual_router_test|_eip/test_del_vip_before_add_eip_to_vm_|0|0|0|1
virtual_router_test|_eip/test_eip_sg_tcp_|0|0|0|1
virtual_router_test|_eip/test_pf_eip_attach_|0|0|0|1
virtual_router_test|_eip/test_vm_ops_|0|0|0|1
virtual_router_test|_eip/test_vm_ops2_|0|0|0|1
virtual_router_test|_lb/test_create_lb_alone_vr_|0|0|0|1
virtual_router_test|_lb/test_create_lb_source_|0|0|0|1
virtual_router_test|_lb/test_create_lb_|0|0|0|1
virtual_router_test|_lb/test_lb_health_check_|0|0|0|1
virtual_router_test|_snapshot/test_create_snapshot_|0|0|0|1
virtual_router_test|_snapshot/test_create_sp_after_delete_|0|0|0|1
virtual_router_test|_snapshot/test_create_image_by_snapshot_|0|0|0|1
virtual_router_test|_snapshot/test_crt_sp_for_both_root_..._|0|0|0|1
virtual_router_test|_snapshot/test_crt_sp_in_stopped_vm2_|0|0|0|1
virtual_router_test|_snapshot/test_crt_sp_in_stopped_vm3_|0|0|0|1
virtual_router_test|_snapshot/test_crt_sp_in_stopped_vm_|0|0|0|1
virtual_router_test|_snapshot/test_delete_snapshot_on_ru..._|0|0|0|1
virtual_router_test|_snapshot/test_crt_sp_in_live_vm_|0|0|0|1
virtual_router_test|_snapshot/test_crt_sp_from_vol_crt_f..._|0|0|0|1
virtual_router_test|_snapshot/test_change_max_sp_depth_l..._|0|0|0|1
virtual_router_test|_regression/delete_sg_with_2_attache..._|0|0|0|1
virtual_router_test|_regression/rm_nic_sg_on_stopped_vm_|0|0|0|1
virtual_router_test|_regression/test_data_volume_with_vm..._|0|0|0|1
virtual_router_test|_vr/test_ping_vms_reboot_vr_|0|0|0|1
virtual_router_test|_vr/test_reconnect_vr_|0|0|0|1
virtual_router_test|_vr/test_start_vrs_simultaneously_|0|0|0|1
virtual_router_test|_host/test_reconnect_host_|0|0|0|1
virtual_router_test|_suite_teardown_|0|0|0|1
local_ps_virtual_router_test|__suite_setup__|0|1|0|0
local_ps_virtual_router_test|_host/test_reconnect_host_|0|0|0|1
local_ps_virtual_router_test|_account/test_vm_ops_by_normal_account_|0|0|0|1
local_ps_virtual_router_test|_account/test_multi_accounts_vm_ops_|0|0|0|1
local_ps_virtual_router_test|_novlan/test_create_vm_|0|0|0|1
local_ps_virtual_router_test|_novlan/test_reboot_vm_|0|0|0|1
local_ps_virtual_router_test|_novlan/test_ping_vms_|0|0|0|1
local_ps_virtual_router_test|_host/test_reconnect_host_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_create_vm_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_reboot_vm_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_snat_create_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_dnat_reboot_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_ping_vms_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_create_3vms_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_start_vm_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_3l3s_vm_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_remove_dhcp_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_2l3_vm_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_crt_vms_with_limited_ips_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_crt_vms_with_limited_ips2.0_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_crt_vms_with_limited_ips2.1_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_start_vms_simultaneously_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_crt_static_ip_vm_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_crt_vm_with_hostname_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_attach_nic_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_attach_nic_stopped_vm_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_attach_nic_reboot_vm_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_chg_static_ip_vm_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_attach_nic_with_static_ip_|0|0|0|1
local_ps_virtual_router_test|_volume/test_add_volume_|0|0|0|1
local_ps_virtual_router_test|_volume/test_add_m_vols_|0|0|0|1
local_ps_virtual_router_test|_volume/test_add_volume_vr_|0|0|0|1
local_ps_virtual_router_test|_volume/test_crt_vm_volume_|0|0|0|1
local_ps_virtual_router_test|_volume/test_crt_vm_2_volumes_|0|0|0|1
local_ps_virtual_router_test|_volume/test_crt_vm_24_volumes_|0|0|0|1
local_ps_virtual_router_test|_volume/test_volume_ops_on_stopped_vm_|0|0|0|1
local_ps_virtual_router_test|_volume/test_crt_vm_from_dvt_|0|0|0|1
local_ps_virtual_router_test|_template_image/test_crt_temp_image_|0|0|0|1
local_ps_virtual_router_test|_template_image/test_del_temp_image_|0|0|0|1
local_ps_virtual_router_test|_sg/test_ingress_|0|0|0|1
local_ps_virtual_router_test|_sg/test_egress_|0|0|0|1
local_ps_virtual_router_test|_sg/test_icmp_|0|0|0|1
local_ps_virtual_router_test|_sg/test_udp_|0|0|0|1
local_ps_virtual_router_test|_sg/test_stopped_vm_|0|0|0|1
local_ps_virtual_router_test|_sg/test_vm_ops_|0|0|0|1
local_ps_virtual_router_test|_sg/test_2vm_egress_|0|0|0|1
local_ps_virtual_router_test|_sg/test_2vm_ingress_|0|0|0|1
local_ps_virtual_router_test|_sg/test_4vm_complx_|0|0|0|1
local_ps_virtual_router_test|_sg/test_detach_l3_|0|0|0|1
local_ps_virtual_router_test|_pf/test_2l3s_pfs_|0|0|0|1
local_ps_virtual_router_test|_pf/test_attach_pf_|0|0|0|1
local_ps_virtual_router_test|_pf/test_delete_pf_after_destroy_vm_|0|0|0|1
local_ps_virtual_router_test|_pf/test_del_vip_before_add_pf_to_vm_|0|0|0|1
local_ps_virtual_router_test|_pf/test_multi_pfs_sg_tcp_|0|0|0|1
local_ps_virtual_router_test|_pf/test_multi_rules_|0|0|0|1
local_ps_virtual_router_test|_pf/test_pf_after_destroy_vm_|0|0|0|1
local_ps_virtual_router_test|_pf/test_pf_sg_tcp_|0|0|0|1
local_ps_virtual_router_test|_pf/test_pfs_tcp_udp_same_vip_ports_|0|0|0|1
local_ps_virtual_router_test|_pf/test_pfs_tcp_with_same_vip_|0|0|0|1
local_ps_virtual_router_test|_pf/test_tcp_|0|0|0|1
local_ps_virtual_router_test|_pf/test_udp_|0|0|0|1
local_ps_virtual_router_test|_pf/test_vm_ops_|0|0|0|1
local_ps_virtual_router_test|_eip/test_2_eips_|0|0|0|1
local_ps_virtual_router_test|_eip/test_2l3s_eip_pf_|0|0|0|1
local_ps_virtual_router_test|_eip/test_2l3s_eips_|0|0|0|1
local_ps_virtual_router_test|_eip/test_attach_eip_|0|0|0|1
local_ps_virtual_router_test|_eip/test_create_eip_vm_|0|0|0|1
local_ps_virtual_router_test|_eip/test_del_vip_before_add_eip_to_vm_|0|0|0|1
local_ps_virtual_router_test|_eip/test_eip_sg_tcp_|0|0|0|1
local_ps_virtual_router_test|_eip/test_pf_eip_attach_|0|0|0|1
local_ps_virtual_router_test|_eip/test_vm_ops_|0|0|0|1
local_ps_virtual_router_test|_eip/test_vm_ops2_|0|0|0|1
local_ps_virtual_router_test|_lb/test_create_lb_alone_vr_|0|0|0|1
local_ps_virtual_router_test|_lb/test_create_lb_source_|0|0|0|1
local_ps_virtual_router_test|_lb/test_create_lb_|0|0|0|1
local_ps_virtual_router_test|_lb/test_lb_health_check_|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_create_snapshot_|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_create_sp_after_delete_|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_create_image_by_snapshot_|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_crt_sp_for_both_root_..._|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_crt_sp_in_stopped_vm2_|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_crt_sp_in_stopped_vm3_|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_crt_sp_in_stopped_vm_|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_delete_snapshot_on_ru..._|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_crt_sp_in_live_vm_|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_crt_sp_from_vol_crt_f..._|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_change_max_sp_depth_l..._|0|0|0|1
local_ps_virtual_router_test|_regression/delete_sg_with_2_attache..._|0|0|0|1
local_ps_virtual_router_test|_regression/rm_nic_sg_on_stopped_vm_|0|0|0|1
local_ps_virtual_router_test|_regression/test_data_volume_with_vm..._|0|0|0|1
local_ps_virtual_router_test|_vr/test_ping_vms_reboot_vr_|0|0|0|1
local_ps_virtual_router_test|_vr/test_reconnect_vr_|0|0|0|1
local_ps_virtual_router_test|_vr/test_start_vrs_simultaneously_|0|0|0|1
local_ps_virtual_router_test|_host/test_reconnect_host_|0|0|0|1
local_ps_virtual_router_test|_suite_teardown_|0|0|0|1
local_nfs_virtual_router_test|__suite_setup__|0|1|0|0
local_nfs_virtual_router_test|_host/test_reconnect_host_|0|0|0|1
local_nfs_virtual_router_test|_account/test_vm_ops_by_normal_account_|0|0|0|1
local_nfs_virtual_router_test|_account/test_multi_accounts_vm_ops_|0|0|0|1
local_nfs_virtual_router_test|_novlan/test_create_vm_|0|0|0|1
local_nfs_virtual_router_test|_novlan/test_reboot_vm_|0|0|0|1
local_nfs_virtual_router_test|_novlan/test_ping_vms_|0|0|0|1
local_nfs_virtual_router_test|_host/test_reconnect_host_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_create_vm_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_reboot_vm_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_snat_create_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_dnat_reboot_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_ping_vms_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_create_3vms_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_start_vm_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_3l3s_vm_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_remove_dhcp_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_2l3_vm_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_crt_vms_with_limited_ips_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_crt_vms_with_limited_ips2.0_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_crt_vms_with_limited_ips2.1_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_start_vms_simultaneously_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_crt_static_ip_vm_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_crt_vm_with_hostname_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_attach_nic_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_attach_nic_stopped_vm_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_attach_nic_reboot_vm_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_chg_static_ip_vm_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_attach_nic_with_static_ip_|0|0|0|1
local_nfs_virtual_router_test|_volume/test_add_volume_|0|0|0|1
local_nfs_virtual_router_test|_volume/test_add_m_vols_|0|0|0|1
local_nfs_virtual_router_test|_volume/test_add_volume_vr_|0|0|0|1
local_nfs_virtual_router_test|_volume/test_crt_vm_volume_|0|0|0|1
local_nfs_virtual_router_test|_volume/test_crt_vm_2_volumes_|0|0|0|1
local_nfs_virtual_router_test|_volume/test_crt_vm_24_volumes_|0|0|0|1
local_nfs_virtual_router_test|_volume/test_volume_ops_on_stopped_vm_|0|0|0|1
local_nfs_virtual_router_test|_volume/test_crt_vm_from_dvt_|0|0|0|1
local_nfs_virtual_router_test|_template_image/test_crt_temp_image_|0|0|0|1
local_nfs_virtual_router_test|_template_image/test_del_temp_image_|0|0|0|1
local_nfs_virtual_router_test|_sg/test_ingress_|0|0|0|1
local_nfs_virtual_router_test|_sg/test_egress_|0|0|0|1
local_nfs_virtual_router_test|_sg/test_icmp_|0|0|0|1
local_nfs_virtual_router_test|_sg/test_udp_|0|0|0|1
local_nfs_virtual_router_test|_sg/test_stopped_vm_|0|0|0|1
local_nfs_virtual_router_test|_sg/test_vm_ops_|0|0|0|1
local_nfs_virtual_router_test|_sg/test_2vm_egress_|0|0|0|1
local_nfs_virtual_router_test|_sg/test_2vm_ingress_|0|0|0|1
local_nfs_virtual_router_test|_sg/test_4vm_complx_|0|0|0|1
local_nfs_virtual_router_test|_sg/test_detach_l3_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_2l3s_pfs_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_attach_pf_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_delete_pf_after_destroy_vm_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_del_vip_before_add_pf_to_vm_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_multi_pfs_sg_tcp_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_multi_rules_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_pf_after_destroy_vm_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_pf_sg_tcp_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_pfs_tcp_udp_same_vip_ports_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_pfs_tcp_with_same_vip_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_tcp_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_udp_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_vm_ops_|0|0|0|1
local_nfs_virtual_router_test|_eip/test_2_eips_|0|0|0|1
local_nfs_virtual_router_test|_eip/test_2l3s_eip_pf_|0|0|0|1
local_nfs_virtual_router_test|_eip/test_2l3s_eips_|0|0|0|1
local_nfs_virtual_router_test|_eip/test_attach_eip_|0|0|0|1
local_nfs_virtual_router_test|_eip/test_create_eip_vm_|0|0|0|1
local_nfs_virtual_router_test|_eip/test_del_vip_before_add_eip_to_vm_|0|0|0|1
local_nfs_virtual_router_test|_eip/test_eip_sg_tcp_|0|0|0|1
local_nfs_virtual_router_test|_eip/test_pf_eip_attach_|0|0|0|1
local_nfs_virtual_router_test|_eip/test_vm_ops_|0|0|0|1
local_nfs_virtual_router_test|_eip/test_vm_ops2_|0|0|0|1
local_nfs_virtual_router_test|_lb/test_create_lb_alone_vr_|0|0|0|1
local_nfs_virtual_router_test|_lb/test_create_lb_source_|0|0|0|1
local_nfs_virtual_router_test|_lb/test_create_lb_|0|0|0|1
local_nfs_virtual_router_test|_lb/test_lb_health_check_|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_create_snapshot_|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_create_sp_after_delete_|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_create_image_by_snapshot_|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_crt_sp_for_both_root_..._|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_crt_sp_in_stopped_vm2_|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_crt_sp_in_stopped_vm3_|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_crt_sp_in_stopped_vm_|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_delete_snapshot_on_ru..._|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_crt_sp_in_live_vm_|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_crt_sp_from_vol_crt_f..._|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_change_max_sp_depth_l..._|0|0|0|1
local_nfs_virtual_router_test|_regression/delete_sg_with_2_attache..._|0|0|0|1
local_nfs_virtual_router_test|_regression/rm_nic_sg_on_stopped_vm_|0|0|0|1
local_nfs_virtual_router_test|_regression/test_data_volume_with_vm..._|0|0|0|1
local_nfs_virtual_router_test|_vr/test_ping_vms_reboot_vr_|0|0|0|1
local_nfs_virtual_router_test|_vr/test_reconnect_vr_|0|0|0|1
local_nfs_virtual_router_test|_vr/test_start_vrs_simultaneously_|0|0|0|1
local_nfs_virtual_router_test|_host/test_reconnect_host_|0|0|0|1
local_nfs_virtual_router_test|_suite_teardown_|0|0|0|1
Total|304|0|4|0|300


I work as a video editor, so I did consider the incorporation of video in web design as a possibility for my historicizing project. I think you have a very exciting topic here and can't wait to see it.


done


See issue #13.


There's another one.
You can sort it with O(nlogn) time and compare adjacent elements. It costs 0 extra space and O(nlogn) time.


Update CacoaPods


I talked to @synctext and this is indeed not possible. Because as I said before the source cannot be validated.


No.
```
l++;
r--;
```


Could also check isEnabled(), but this is safer and directly addresses the NPE


Closes #7


relase ---> release


Removed demo usage of colours and fonts in typography to prevent the inevitable lint errors when editing the variables.


"one to one" ---> one-to-one


Checkmarx scan results 06 Oct 2015 15:57 UTC +02:00

High
Medium
Low
Info

New
6
33
399
0

Recurrent
0
0
0
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

0deb47d4599928e0615ba250d5c5eec61069afc3	@amitchen22


Awesome :trophy:


I know, these things are hard to phrase: I'm always trying to find a nicer way...

"allow to" ---> "optionally"


Checkmarx scan results 06 Oct 2015 16:00 UTC +02:00

High
Medium
Low
Info

New
14
12
2
29

Recurrent
0
0
0
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

a6f44e41cbec199739cdda9aba94708fd0a2405a	@amitchen22


logs ---> log OR "log file"


@miigotu so where should I add it?


Mierda: ceylon/ceylon-ide-eclipse#1444.


Perhaps: "accept a higher [...]"


There is a typo in the commit message


@levicole there's probably a cleaner way to implement this but I haven't figured it out as of yet.


WebView想到方法了。。很简单，在Model的Parse方法里面添加HTML数据的style，等下改好给你看看。。。BTW，那个Behavior的我设置成了我的那个，我是只要往下滑，FAB就会消失掉，再也不会出来。。。（可能这个比较符合逻辑？说明不再需要看加载进度了？）。不知道这样可不可以。。。


Waait a minute..
Wasn't it always 24 hour time?


* This is the parent node of the main swapper images
* When the main images rotate, we set `absolute positioning` to them for them to move relative to their closest __`positioned`__ ancestor
* So we set this direct parent as `relative positioning` so that those main images move relative to this parent


有一点小小的变化，也是我一直纠结的地方：新版本的 ->login() 默认是延迟执行登录的，也就是方法会立刻返回，然后再在适当的时候执行（在加载完所有插件之后） 老版本的login是阻塞式的，登录完才执行结束，可以通过->login(delay=>0)来实现老版本的方式，我觉得这块我设计的比较糟糕 还没想好怎么样才好


just leave it for now. We have to make some rearranging to accomodate a different user agent for each provider, or calculate a REAL user agent to use.


Looks fine to me.


If you replace buttons with a, then everything is going to break. If you want it to work in firefox with a elements, so you gotta rewrite whole toolbar because there are multiple absolute elements that are not relative to their parent


PostQRcode这个插件不需要在最开始use；直接load即可


A cursory check indicates that everything is in order. I can merge this in once others have taken a look.


Does this change really break existing code out there?


I much prefer the new version, visually.
The contrast between text and bg being small,
I'd prefer the inactive tabs to be less dark.
Not sure how that affects the default palette.
Btw. where do I see any `tc-tab-divider`?


Most scroll bars have two simple functions: to move through a document and indicate where readers are at in the document. Despite these simple tasks, I agree that the scroll bar has a strong impact on how we view text via the web. It's a design that I never really considered until I read Haas' study of it.


Adding filter here


You forgot to remove the MIT license.


delay=>1 是login中的参数 不是new中的, 而且->login() 默认就是delay=>1


* This `.animation()` is set available by the `ngAnimate` module
* If a module (`erAnimations` in our case) has `ngAnimate` as a dependency, it can register JS animation by using `erAnimations.animation()`
* The first argument to this method is the selector (normally a __class__ selector) of the elements we want to animate


Well, that doesn't seem wrong.


This commit is being reviewed as a part of review issue #63.


What elements are you talking about? There shouldn't be any elements without a correct root. Maybe something else is broken.


Could you use the same `.. note::` syntax that we are otherwise using? :)


@nrgaway can you explain for what this commit is here? And why I've merged it? ;)

This cause uncommited changes in `debian/changelog` in source repository, just because package was built. It is easy to not notice it and commit when using `git commit -a`. Also it makes `make check` (check for uncommitted changes) pretty useless.


Якщо оголошення і використання відбавається у одній області видимості, не треба їх розривати. Коли спершу все нагорі оголошено, а потім йому присвоюються значення, таке важче сприймати


Временно


Not that I'm happy about it. I'll find a better fix.


you switched this to a `Nan::ObjectWrap` but left the `node::ObjectWrap` in `nlv_object.h`. Is that intentional? If not, I'm a little concerned about why these changes don't break anything (and what I _should_ be looking for in that case)


Should be 'fi' not 'esac'


Both forms, when submitted, have their value picked from a single method `TopicsController#successor` which prioritises a page over a (sub)topic. Do we need two forms? Would it not be clear with one button saying "Archive and redirect"?


show common used emojis, when use input a single ':'

![emoji_top10](https://cloud.githubusercontent.com/assets/3104826/10311439/4320ead0-6c79-11e5-98f5-69f39797d775.png)


I think this cannot work since originally `$config` is not transfered to `preMap` method by reference.


Nevermind, merged and away! :+1:


Here what's gonna happen if I replace buttons with a:
![image](https://cloud.githubusercontent.com/assets/8620461/10311431/a4c5b91a-6c14-11e5-9b73-bbcb6a31cf54.png)

![image](https://cloud.githubusercontent.com/assets/8620461/10311454/bf63a8e0-6c14-11e5-977e-0092abf041a6.png)

It is not fitting correctly, and I bet that <div class="ember"> is responsible for that, it has 0 height.


Disclaimer: I have only limited understanding of config parser, so I may be completely off track here.


Is this causing issues with the android compile on jenkins? I'm getting an error, referring to MapgenFractal, when github tries to check a commit on a pull request I made.
https://jenkins.unix-experience.fr/job/minetest-Android-OfficialRepository/1463/console


All good!


yooooooo these 2 buttons are :fire::fire::fire:


Note that you could simplify that class a little more by sublassing `Command` directly, as I suggested in b05b0c2.

IMO, sublassing `setuptools.command.test` doesn't make much sense, as it seems to be specific to `unittest`, and just requires some additional boilerplate.


Awesome commit


`addClass` and `removeClass` will call this callbacks with 3 arguments:
* `element`
 * Can be a `jQuery` element if `jQuery` is found
* `className`
 * The className to be added or removed
* `done`
 * The purpose of `done` is so you can let Angular know when the JavaScript animation has ended by calling it


LGTM ;)


Already done.


`element` is already a `jQuery` element.


I see what you mean. The caret itself seems to be broken. I will take a second look. Sounds like your solution is the only way to go.


Way to go, Chad!


До слова, `scanner.nextInt()` теж працює нормально, і зразу повертає число (робить `parseInt` під капотом)


Ignore other class changes.


Changed this only.


А отут у вас повзе форматування. `Ctrl+Alt+L`, якщо не помиляюсь.


刚洗澡去了。。。
我的是下滑会消失，上滑再出现，但居然会不定期出现不起作用的情况。。。。我也是比较蛋疼。。。。
可以先按你的啊，毕竟这是一个尝鲜，人家不喜欢的话，大不了再换回普通的 progressbar 。。。
（亏我这几天还在看前端，居然连改 style 都没想到。。。）

2015-10-06 22:14 GMT+08:00 Troy Liu <notifications@github.com>:

>
> WebView想到方法了。。很简单，在Model的Parse方法里面添加HTML数据的style，等下改好给你看看。。。BTW，那个Behavior的我设置成了我的那个，我是只要往下滑，FAB就会消失掉，再也不会出来。。。（可能这个比较符合逻辑？说明不再需要看加载进度了？）。不知道这样可不可以。。。
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/tangqi92/BuildingBlocks/commit/d3e731d253592de576ec801c1e9714dbb37e9afb#commitcomment-13615847>
> .
>


Good catch @benjaoming - @djallado spotted some minor changes in the install and release notes docs, so he should  be able to use the `.. note::` syntax.  Yes, these were thru the #4570 PR.


:open_mouth: hell yeah they are


- You don't have to upload this file


* `addClass` and `removeClass` each returns a function
* This is an optional function that's called when the animation is cancelled (when another animation takes place on the same element) as well as when the animation has completed
* A boolean parameter is passed into the function which lets the developer know if the animation was cancelled or not
* This function can be used to do any cleanup necessary for when the animation finishes


- Bin folder shouldn't be committed


@iconexperience I hope you don't my refactoring tricks. This one compressed better, and is still readable I think.


> Btw. where do I see any tc-tab-divider?

The tab divider is the horizontal line between the tab buttons and the content (it's a vertical line for the vertical tabs)


LGTM


Cannot use `img-responsive` anymore as we need fixed image size.


Look at next patch that makes use of frandom instead.


This commit is being reviewed as a part of review issue #63.


Do you think we could find something better for this?
You could inject a custom `org.hibernate.search.engine.service.spi.Service`, retrieve it from the `serviceManager` as on line 63.
A `Service` would be global, so I'm not sure if it would be of much help :-/


TODO: Move Locked to after "spam" for consistency


We load all images in this main image section and use `active` class to mark the "current" image.


TODO Move to below the `location == 'spam'` check.


`throw new UnsupportedOperationException(...)` instead?


looks great ;)


Heh, I wouldn't say this is a *horrible* idea, it just loses some features. For example, I don't usually just search for a simple string, I use the terms: http://notmuchmail.org/manpages/notmuch-search-terms-7. Typically I'm happy with the results.


I found the following solution from this thread: https://github.com/boto/boto3/issues/134 near the bottom

```
import boto3

from botocore.handlers import disable_signing

resource = boto3.resource('s3')

resource.meta.client.meta.events.register(
    'choose-signer.s3.*', disable_signing)
```


Not Python 3 compatible. See http://python3porting.com/problems.html "Unorderable types"


this is where I'd rather override the code from Search which would normally parse for the `sharding_strategy`


TODO: move to after 'spam'


Думка щодо обмеження області знизу і зверху — правильна.

А от вибирання довільного числа у цій області — не найефективніший спосіб. Цілком імовірна така ситуація, що одного разу програма запитуватиме усі числа по порядку (0, 1, 2,...). Більше того, цілком імовірна і така ситуація, що програма запитуватиме одне і те саме число підряд багато разів.

Подумайте, як було би правильніше і ефективніше вирішити цю задачу.


Don't you need to test here to see if the F08 bindings were requested?  E.g., what if OMPI_TRY_FORTRAN_BINDINGS was set to only build MPIFH?


hola esto está bacan


Nervermind previous comment, maybe is it better to change the interface to something like `isUsed` instead of `count`?


TODO: fix formatting + move below L362


@agrrh- I just ran into the same thing.  Filed this issue #12648


The date was being used originally so that the same question on the same day always had the same answer. I did this because people would ask a question, then ask the same question again to get a different answer. So I added that to ensure its always the same answer (unless its a different day)


At first I thought it would be that but then I wondered why it would be a gradient.


I will keep the actual script until we get a solution for this http://paste2.org/aCkMmFmx
It crashes the server, and even with target=1 the npc is not summoned.


saw approval from hmmr
at https://github.com/basho/riak_kv/commit/51d06ef8ae29e2ee6e42672efd28782e8b94f3a7


merging basho/riak_kv/feature/parsing_error_messages = 51d06ef8 into borshop-integration-1206-feature/parsing_error_messages


basho/riak_kv/feature/parsing_error_messages = 51d06ef8 merged ok, testing candidate = c73335a9


TODO: Fix formatting + move to below L415


Ah, the idea was for it to fade away off to the right (and bottom).


yesssssss


Not intentional, I will switch that to `Nan::ObjectWrap` as well. Those missed spots will not break anything as `Nan::ObjectWrap` is simply an API sugar/wrapper over `node::ObjectWrap`, it doesn't change its data layout. It also compiles well cause the types get lost in the explicit pointer casting through libuv but us a I said, `Node::ObjectWrap` and `Nan::ObjectWrap` is the same object in memory.


Are you sure this is an "unexpected" error? Doesn't this happen every time a part is not stored in the storage? This should be pretty normal I think.


It is better to use prepare statement that directly use strings to create the query


The `if`/`else` branches seem to be inverted to me. If the `String` is frozen, use it as a key, if it is not frozen dup & freeze and use the obj as a key.


October 9,2015 7:00-7:30 IST
Mohd Mujahed


TODO: add image and listing under moderation tools


Me equivoqué al hacer click y quedó el log del commit anterior. Este commit fueron solo cambios PEP.


In your code please change
`from DatasetUtils import xyz` to
`from knossos_python_tools.dataset_utils import xyz`


@lbianc  Yes but like i said there is a problem i will have to lower all instruction who uses VptrToReg. How i can lower vret and storesd instructions? These instructions call asm instruction which not have a vasm. I don't like the idea to create other vasm instructions just for that. See attach
[vasm.txt](https://github.com/PPC64/hhvm/files/8580/vasm.txt)


> you switched this to a Nan::ObjectWrap but left the node::ObjectWrap in nlv_object.h

Okay, I actually don't see the place where I missed it, could you point me?


Ah yeah you're right - thanks


Move to L792 and change "Submissions" to "Things"


LGTM - sorry for the delay (I did not notice this PR)


The GNU recommendations are here:
https://www.gnu.org/licenses/gpl-howto.html
and include this line:

>You should also include a copy of the license itself somewhere in the distribution of your program. All programs, whether they are released under the GPL or LGPL, should include the text version of the GPL. In GNU programs the license is usually in a file called COPYING.

So I think, revert it and put the full text back.

The other thing the howto says is that every source module should contain,

>a copyright notice (such as “Copyright 1999 Terry Jones”), and a statement of copying permission, saying that the program is distributed under the terms of the GNU General Public License...

This is true, yes? I believe that these lines are the real legal protection and more important than where the license text is.


Same here, this is a huge bug


Actually, I'm somewhat confused as to what this configury does.

1. If the user has requested something that we can't build, we should AC_MSG_ERROR (not WARN).
1. If the user specifically didn't ask for mpi_f08 (e.g., they disabled fortran, or are only building mpifh, ...etc.), then we don't need to warn about not being able to build mpi_f08
1. If the user didn't ask for anything (but didn't disable fortran), then it is reasonable to warn that configure is choosing not to build mpi_f08 because the profiling layer is not being built.

Is the last item on that list the purpose of this new configury?  If so, does this new configury also address the other to items on this list?


Read carefully, this happens only if there is an error which is `!os.IsNotExist(err)`


This is a super hack, and nearly the same as me simply declaring:

```
document = window.document;
$ = window.jQuery;
```

etc without the `var` keyword. Difference is that I won't be overriding any existing variables because I'm throwing them in the prototype.

`__proto__` is deprecated in ES5 which is why this code is probably smelly, but in ES6 it's legit! https://github.com/jshint/jshint/issues/2371. Looks like `Object.setPrototypeOf()` is also a thing? http://www.ecma-international.org/ecma-262/6.0/index.html#sec-object.setprototypeof


I simple don't like the switch and the enum class declared inside the function. This way makes the code a little smaller.


No.


The site does currently load MailChimp's JS (from list-manage) when you click the sign up. This is not technically true in that case.


Can you write a bug on pyffi


DoMeleeAttackIfReady - missing.
Ну да ладно!


It's pretty close, though it seems kebabCase splits camel-cased words while slugify doesn't.


I've never seen such bomb-ass buttons :bomb:


@vivekbarsagadey, no rush. I just wanted to make sure you were aware of my questions. Take your time, and I look forward to seeing your responses. Thanks!


Remember to only include relevant code in each commit. This should have been its own commit with a message lik "Remove debug code". Although one should try to not commit debug code in the first place.


Right, I was talking yesterday with Roberpot with this in IRC and we reached that same conclusion: moving comments is not enough to generate good docs. He's working on it right now :)


https://github.com/Rush/node-libvirt/blob/master/src/nlv_object.h#L36
It's `Nan::ObjectWrap` in my branch


let me know what happens! :)


When is it ever a `ValueError`?


arg sorry there are so many commits and the patchset is so large I seem to have missed that, carry on!


The manual says that we accept v == 1.0


OK.  Perhaps, that was my mistake.


Remember, if possible, split commits into small changes, not lists of changes. Makes it easier to see what has been changed, and why.


Thanks @kkoopa. I was working on moving `node-libvirt` to nan2 and had some troubles but finally I made it work. https://github.com/hooklift/node-libvirt/pull/62


Are the names of the model correct here? Do you mean Waypoint and WaypointLocale?


i think i need one more comment - i think i'm not officially "participating" until i've commented on this commit, even though i wrote it :(


No, it’s just our mail being slow.
On Oct 6, 2015, at 10:47, Dmitriy Likhten <notifications@github.com<mailto:notifications@github.com>> wrote:

let me know what happens! :)

—
Reply to this email directly or view it on GitHub<https://github.com/jdpopkin/email_reply_parser/commit/bf9d5ce00f52f9096acf87a60f3ed6134338f03e#commitcomment-13616669>.


looks reasonable, I'll get back to properly working on this branch when the `ghcjs-base` library things are done.


Well, thank you :) but without the codebase that has already been tested and debugged for years, this would have been a difficult task... and without fishtest, how would we test? Some credit to all of us!


Ты ж уже отослал did create group chat dialog - зачем следом отправлять did update?


No problem, and actually I think I am done.


This will return the value from sm_config eveb when the SR is not thin vlhd


ref #285


OK, I got it. Here's the modification version:
	public String str_compress(String a) {
		StringBuilder ans = new StringBuilder();
		int len = a.length();
		int dist;
		for (int i=0; i<len; ) {
			dist = 1;
			while (i + dist < len && a.charAt(i) == a.charAt(i + dist))
				dist ++;
			ans.append(a.charAt(i));
			ans.append(dist);
			i += dist;
		}
		if (ans.toString().length() < a.length()) return ans.toString();               //modified
		else return a;                                                                                //modified
	}


Typo: `incremented`.


didCreate не отсылает месседж "has added Username to the group chat"


I don't mind at all, I even linke them.

By the way, do you know what the `|| 0` in <a href="https://github.com/paperjs/paper.js/blob/boolean-fix/src/path/Curve.js#L1446">`Curve.js lines 1446/1447`</a> is for?


This isn't really a config, is it? I'd probably just call it 'RouterData' or something (a better name is welcome :)), or even `R` :-o.


Oh yes, I mean `Document` and `DocumentLocale`.


You maybe right about the fact that the os.path is not required, I did not try without. But I well included it at start. If You study the current python manuals , they all state it must be done like that, I just followed those manuals.

Yes on enigma2(pli4) all is based on hotplug . Whitout hotplug it will not work anymore. If some images non pli4 based are made, Developers should develop there own way to pass to enigma2 that a change happened.  This whole bunch is for pli4 , does require a correct mdev and bdpoll. Also the mdev.conf must be right.

I'm the first which implemented the use off audiocd whitout the need off a cdfs pseudo file system, which let think all other software that we do have a file system. Which was not the case. The cdfs modules have been removed from linux kernels since ..... very long. What was used where patches on old removed base linux kernel software (module based so quit high or low .... level) . Those patches where unstable and hacks , the longer the more unstable . And actually even not really used. Since on a stb audio cd's where always played trough gstreamer using libcdio.....

The step off using cdfs is really not needed and deprecated. But we still needs a base playlist made for audio cd's , only to have the info play uri=cdda://<xx>  to gstreamer.


А ты уверен что completion ne nil?


looks like there's an extra space in front of def_bool?


It might make sense to build the docs periodically and put it on some webspace - let me know when it is time to setup a cronjob for running Doxygen.


Should empty category not go to the Admin to be assigned a category?? Auto will select Admin


Does it do that for you? I don't think I've been seeing a gradient, neither horizontally nor vertically.


FYI, ES7 introduces an reserved word `async`. You might want to rename the variable.


Yes they will.  Unfortunately while testing this out it appears to have issues with re-centering the dialog that was in the last PR but reverting those changes also doesn't appear to help.  This commit will be amended once I find the right way around the centering issues so the rename doesn't completely break everything.


here code is written ????


Since `yield` returns cursor and not future, I suggest you rename `f1` variable to be called `cursor`.


testing by another user


If the value does not contain a ``T``, then this line https://github.com/key/isodate/blob/master/src/isodate/isodatetime.py#L48 will raise a ``ValueError``. And the ``parse_date`` function can raise the ISO error later on.


The role of `ArchiveDocument` and `HistoryMetaData` is not clear.
Maybe you could add a short hint.


I merged in `master` yesterday and took a look at this again.  I'm pretty concerned about performance - I ran a couple queries on prod, and they were very slow - this table is massive.


@richardpl
Did you port af_karaoke to lavfi?


yep, that's good and bad think on python, it doesn't care about types :smile:


What do you need the "full_case parallel_case" for?


了解しました。確かにセーブ・ロードのによって若干見た目を変える必要が出る可能性もありますね。
ただし、その際はフラグで処理分けをさせるとあちこちにifが必要となるため、継承を利用すると良いと思われます。


OK, I'll change it...


Yeah, that's what i meant.


Oups


I also didn't like `Cfg` but I couldn't find anything meaningful :-|
I like `RouterData`, `R` is to simplistic :)


Is it ok? I just new to python :wink:


Your change at line 177 will never kick in...
Did you forgot to change the o["Threads"] too ?
Or submit test with Hash=128 Threads=2 ?


А теперь я понял, конечно, надо было бы эту ситуацию в сервисах обработать, чтобы в системных сообщениях сразу уходил нужный текст... но пока давай оставим как ты сделал.


all tests pass:
success: http://buildbot.bos1/builders/test-riak_kv/builds/2461


Sent another commit


This commit is being reviewed as a part of review issue #63.


Out of curiosity: what "_in" stands for in "document_in"?


genuinely curious if you see an advantage in using the `class` syntax. i've seen many "js pros" denounce the use and/or underlying implications that `class` introduces in a language like js, but i'm interested in your personal experiences/opinions, especially with regard to real world libraries like react-router, overall interoperability, dealing moving targets like react, etc.


Just having a problem with these learnobject as soon as it's going live.. like i cant access to my database and no errors found..


Thank you. I'll look into this.
> Also someone explain to me how to code tag on here please!

Just use [Markdown syntax](https://guides.github.com/features/mastering-markdown/#GitHub-flavored-markdown) :smile:

    Four spaces to intend code automatically
You can have the exact same result by bracketing your block of code with ```

or
```
use `your inline code` to have the following result
```
use `your inline code` to have the following result


8:00 - 6:00


А почему ты тут убрал забор истории - он где-то в другом месте делается?


The documents that comes *in*. ;)


Do you mean DocumentVersion and HistoryMetaData?


p.s.
and about os.path import?
MiLo changed it (by a lot off modules) . Strictly speaking I think You and MiLo have right.
But ...... python manual tells both are needed. And I guess manual is wrong on that point but.....


Probably it would be good to add a TODO or a message


Да, он делается после логина в чат в диалог дата сорсе


They're supposed to download a .zip uploaded by one of us (not working because my Edge and GitHub don't mesh well), not from the commit list itself.


GitHub :smile: was removed.


@onebone What? Please merge the changes in branch php7 https://github.com/onebone/EconomyS/pull/164


I think there one more issue remaining - if pending connections resolve to be dead, then the peding futures are never resolved. I.e.
1. You request connection
2. Since there are pending connection, future is put to waiting queue. The future is returned to the caller
3. All pending connections resolve as dead
4. Caller's future is never resolved

You can see this if you comment out `self.start_proxy()` line in your second test or if you use `bad_dsn` in your first test.

If think the solution would be to fix `ConnectionContainer.add_dead` to check whether all connections are dead and if they are, to call `abort_waiting_queue`.

Also I think that `all_dead` property should take pending connections into account.

What do you think?


I think this should be the following :wink:

```java
super.onRequestPermissionsResult(requestCode, permissions, grantResults);
```

PS: Thanks for the great project!


Students/Others/Pop I. Andrei/Assignment 1 - Java/src/testProject/MultiplesSum.java

there is a much more elegant solution to this. you can find it here:
http://math.stackexchange.com/a/9305


I have submitted the test at 2 threads. It should kick in.


True that anything specific to SQL-Ledger is cruft that can and should be removed.  The !$INTEGRATED_AR (note the negation) condition includes this.


OK, makes sense


Solltest Coroutines benutzen statt der update, da kannst auch mit ner yield warten, bis das löschen fertig ist zum beispiel mit yield return startcoroutine(methodezumlöschen());


Close #18


Нужно использовать информативные названия констант. Например  START_QUERY, SET_LOCATION, STREAM_MODE.


this seems off in the TestVeryFragmented


argh, sorry, I forgot that one :)


This is probably the ugliest list comprehension I've seen, but ok.


dfdswss


e con eso que se supón que se fai?


Ja du führst checktop in checktop aus durch refresh das funktioniert nur mit coroutines


it's better to not hardcode faf modules, but rather pass them to task from settings.json during task invocation


It's getting better with the is_field property i'm adding


Why not implicit `return None`?


@skalk: please add the last 4 commits of https://github.com/alex-ab/genode/tree/intel_kms to your intel_kms branch - it contains some fixups for the platform driver


Just to be clear, keep code that is within a "if ($INTEGRATED_AR)" condition.  Remove code that is in the "else" condition or equivalent.  I guess that's what you mean, just want to make sure.


Don't worry. They were revoked an hour after the commit :) But thanks for pointing that out, @timfallmk !


I like being explicit


fast-forwarding end-to-end/timeseries to borshop-integration-1206-feature/parsing_error_messages = c73335a9


This commit is being reviewed as a part of review issue #63.


Можно было использовать "JCommander", который все сделает просто и безопасно.


You meant `pattern`?


From https://irc.gitter.im when I type `/join #mytardis/mytardis` I get "Invalid channel or insufficient permissions". I heard about this project at http://irclog.perlgeek.de/crimsonfu/2015-10-06


Another way to perform the operation without another tmp register: Replace shift by addq operations. This will emit up to 8 add instructions but it will work without uses another tmp register.

```C++
    // Calculate index*scale type address.
    uint8_t scale = s.scale;

    if(!s.index.isValid() && !s.base.isValid() &&
       !ignore_base && s.disp == -1) {
      assert(false && "Invalid address mode");
    }
    // direct address usign copy instruction
    if((s.base.isValid() &&
       !ignore_base) && s.disp == -1 && !s.index.isValid()) {
      emit(copy{s.base, d});
    }

    // load displacement to register first
    if(s.disp > -1) {
      emit(ldimmq{s.disp, d});
    }
    if(s.index.isValid()) {
      if(s.disp > -1) {
        while(scale) {
          emit(addq{s.index, d, d, VregSF{0}});
        }
      } else {
        emit(shlqi{shift, s.index, d, VregSF(0)});
      }
    }
    if(s.base.isValid() && !ignore_base &&
    (s.disp > -1 || s.index.isValid())) {
      emit(addq{s.base, d, d, VregSF(0)});
    }
    emit(load{*d, d});
  }
```


Frankly saying, I cannot reproduce the compilation error you describe. Please post the error message along with what wxWidgets and compiler version you're using.

`kademlia/kademlia/SearchManager.h` is completely unrelated to amuleweb and the resolution of this compile error should not include it.


a little description of the test ? :-)


Submit this as PR and I'll merge in!


I just removed the if condition not the code below it


Hi,

Sorry for the delay, got caught up in other stuff.

Regarding this particular fix, my main issue with the onPlayerError path is that errors are fatal to the stream play. In my opinion every recoverable error in codec should be caught and recovered as low down as possible.

In my use case, I have an HLS stream that causes the aac decoder to fail on quality change with the following stack trace:
```
10-06 16:51:22.459 16585-9000/com.google.android.exoplayer.demo E/ACodec: [OMX.google.aac.decoder] ERROR(0x80001001)
10-06 16:51:22.459 16585-9000/com.google.android.exoplayer.demo E/ACodec: signalError(omxError 0x80001001, internalError -2147483648)
10-06 16:51:22.459 16585-9000/com.google.android.exoplayer.demo E/MediaCodec: Codec reported err 0x80001001, actionCode 0, while in state 6
10-06 16:51:22.469 16585-8986/com.google.android.exoplayer.demo E/ExoPlayerImplInternal: Internal runtime error.
10-06 16:51:22.469 16585-8986/com.google.android.exoplayer.demo E/ExoPlayerImplInternal: java.lang.IllegalStateException
10-06 16:51:22.469 16585-8986/com.google.android.exoplayer.demo E/ExoPlayerImplInternal:     at android.media.MediaCodec.native_dequeueOutputBuffer(Native Method)
10-06 16:51:22.469 16585-8986/com.google.android.exoplayer.demo E/ExoPlayerImplInternal:     at android.media.MediaCodec.dequeueOutputBuffer(MediaCodec.java:1033)
10-06 16:51:22.469 16585-8986/com.google.android.exoplayer.demo E/ExoPlayerImplInternal:     at com.google.android.exoplayer.MediaCodecTrackRenderer.drainOutputBuffer(MediaCodecTrackRenderer.java:802)
10-06 16:51:22.469 16585-8986/com.google.android.exoplayer.demo E/ExoPlayerImplInternal:     at com.google.android.exoplayer.MediaCodecTrackRenderer.doSomeWork(MediaCodecTrackRenderer.java:458)
10-06 16:51:22.469 16585-8986/com.google.android.exoplayer.demo E/ExoPlayerImplInternal:     at com.google.android.exoplayer.ExoPlayerImplInternal.doSomeWork(ExoPlayerImplInternal.java:431)
10-06 16:51:22.469 16585-8986/com.google.android.exoplayer.demo E/ExoPlayerImplInternal:     at com.google.android.exoplayer.ExoPlayerImplInternal.handleMessage(ExoPlayerImplInternal.java:213)
10-06 16:51:22.469 16585-8986/com.google.android.exoplayer.demo E/ExoPlayerImplInternal:     at android.os.Handler.dispatchMessage(Handler.java:98)
10-06 16:51:22.469 16585-8986/com.google.android.exoplayer.demo E/ExoPlayerImplInternal:     at android.os.Looper.loop(Looper.java:155)
10-06 16:51:22.469 16585-8986/com.google.android.exoplayer.demo E/ExoPlayerImplInternal:     at android.os.HandlerThread.run(HandlerThread.java:61)
10-06 16:51:22.469 16585-8986/com.google.android.exoplayer.demo E/ExoPlayerImplInternal:     at com.google.android.exoplayer.util.PriorityHandlerThread.run(PriorityHandlerThread.java:40)
```

with this commit, the IllegalStateException in dequeueOutputBuffer is caught and the stream resumes immediately without even any audio "click". Unfortunately the stream in question is an internal stream and I cannot publish its URL.

Or did I miss something and there is another way to achieve the automatic recovery of the audio decoder?


If in need, you can also use all Array methods, because there's a polyfill (arr.forEach..), so that we depend less on jQuery


@blesh This was reason of `ArrayObservable` ctor behaves way as mentioned below. You can say I did dumb mistake. :(


Nice catch! I didn't even notice the warnings in the Vorbisfile API docs for the Windows platform ;)


Awesome to see that you're using this as a study guide! 👍🏻👍🏻


Code doesn't compile in the arduino IDE.

Arduino: 1.6.5 (Windows 8.1), Board: "Arduino/Genuino Uno"

Build options changed, rebuilding all

Using library SPI in folder: C:\Users\ladmin\Downloads\arduino-1.6.5-r5-windows\arduino-1.6.5-r5\hardware\arduino\avr\libraries\SPI

Using library Ethernet in folder: C:\Users\ladmin\Downloads\arduino-1.6.5-r5-windows\arduino-1.6.5-r5\libraries\Ethernet

Using library PubSubClient in folder: C:\Users\ladmin\Documents\Arduino\libraries\pubsubclient-master

Using library Wire in folder: C:\Users\ladmin\Downloads\arduino-1.6.5-r5-windows\arduino-1.6.5-r5\hardware\arduino\avr\libraries\Wire

Using library SD in folder: C:\Users\ladmin\Downloads\arduino-1.6.5-r5-windows\arduino-1.6.5-r5\libraries\SD

Using library FTOLED in folder: C:\Users\ladmin\Documents\Arduino\libraries\FTOLED-master

C:\Users\ladmin\Downloads\arduino-1.6.5-r5-windows\arduino-1.6.5-r5\hardware\tools\avr/bin/avr-g++ -c -g -Os -w -fno-exceptions -ffunction-sections -fdata-sections -fno-threadsafe-statics -MMD -mmcu=atmega328p -DF_CPU=16000000L -DARDUINO=10605 -DARDUINO_AVR_UNO -DARDUINO_ARCH_AVR -IC:\Users\ladmin\Downloads\arduino-1.6.5-r5-windows\arduino-1.6.5-r5\hardware\arduino\avr\cores\arduino -IC:\Users\ladmin\Downloads\arduino-1.6.5-r5-windows\arduino-1.6.5-r5\hardware\arduino\avr\variants\standard -IC:\Users\ladmin\Downloads\arduino-1.6.5-r5-windows\arduino-1.6.5-r5\hardware\arduino\avr\libraries\SPI -IC:\Users\ladmin\Downloads\arduino-1.6.5-r5-windows\arduino-1.6.5-r5\libraries\Ethernet\src -IC:\Users\ladmin\Documents\Arduino\libraries\pubsubclient-master\src -IC:\Users\ladmin\Downloads\arduino-1.6.5-r5-windows\arduino-1.6.5-r5\hardware\arduino\avr\libraries\Wire -IC:\Users\ladmin\Downloads\arduino-1.6.5-r5-windows\arduino-1.6.5-r5\libraries\SD\src -IC:\Users\ladmin\Documents\Arduino\libraries\FTOLED-master C:\Users\ladmin\AppData\Local\Temp\build1805392241913835450.tmp\SecuritySensorToMQTT.cpp -o C:\Users\ladmin\AppData\Local\Temp\build1805392241913835450.tmp\SecuritySensorToMQTT.cpp.o

SecuritySensorToMQTT.ino:113:43: error: no matching function for call to 'PubSubClient::PubSubClient(byte [4], int, void (&)(char*, byte*, int))'
SecuritySensorToMQTT.ino:113:43: note: candidates are:
In file included from SecuritySensorToMQTT.ino:36:0:
C:\Users\ladmin\Documents\Arduino\libraries\pubsubclient-master\src/PubSubClient.h:99:4: note: PubSubClient::PubSubClient(const char*, uint16_t, void (*)(char*, uint8_t*, unsigned int), Client&, Stream&)
    PubSubClient(const char*, uint16_t, MQTT_CALLBACK_SIGNATURE,Client& client, Stream&);
    ^
C:\Users\ladmin\Documents\Arduino\libraries\pubsubclient-master\src/PubSubClient.h:99:4: note:   candidate expects 5 arguments, 3 provided
C:\Users\ladmin\Documents\Arduino\libraries\pubsubclient-master\src/PubSubClient.h:98:4: note: PubSubClient::PubSubClient(const char*, uint16_t, void (*)(char*, uint8_t*, unsigned int), Client&)
    PubSubClient(const char*, uint16_t, MQTT_CALLBACK_SIGNATURE,Client& client);
    ^
C:\Users\ladmin\Documents\Arduino\libraries\pubsubclient-master\src/PubSubClient.h:98:4: note:   candidate expects 4 arguments, 3 provided
C:\Users\ladmin\Documents\Arduino\libraries\pubsubclient-master\src/PubSubClient.h:97:4: note: PubSubClient::PubSubClient(const char*, uint16_t, Client&, Stream&)
    PubSubClient(const char*, uint16_t, Client& client, Stream&);
    ^
C:\Users\ladmin\Documents\Arduino\libraries\pubsubclient-master\src/PubSubClient.h:97:4: note:   candidate expects 4 arguments, 3 provided
C:\Users\ladmin\Documents\Arduino\libraries\pubsubclient-master\src/PubSubClient.h:96:4: note: PubSubClient::PubSubClient(const char*, uint16_t, Client&)
    PubSubClient(const char*, uint16_t, Client& client);
    ^
C:\Users\ladmin\Documents\Arduino\libraries\pubsubclient-master\src/PubSubClient.h:96:4: note:   no known conversion for argument 3 from 'void(char*, byte*, int) {aka void(char*, unsigned char*, int)}' to 'Client&'
C:\Users\ladmin\Documents\Arduino\libraries\pubsubclient-master\src/PubSubClient.h:95:4: note: PubSubClient::PubSubClient(uint8_t*, uint16_t, void (*)(char*, uint8_t*, unsigned int), Client&, Stream&)
    PubSubClient(uint8_t *, uint16_t, MQTT_CALLBACK_SIGNATURE,Client& client, Stream&);
    ^
C:\Users\ladmin\Documents\Arduino\libraries\pubsubclient-master\src/PubSubClient.h:95:4: note:   candidate expects 5 arguments, 3 provided
C:\Users\ladmin\Documents\Arduino\libraries\pubsubclient-master\src/PubSubClient.h:94:4: note: PubSubClient::PubSubClient(uint8_t*, uint16_t, void (*)(char*, uint8_t*, unsigned int), Client&)
    PubSubClient(uint8_t *, uint16_t, MQTT_CALLBACK_SIGNATURE,Client& client);
    ^
C:\Users\ladmin\Documents\Arduino\libraries\pubsubclient-master\src/PubSubClient.h:94:4: note:   candidate expects 4 arguments, 3 provided
C:\Users\ladmin\Documents\Arduino\libraries\pubsubclient-master\src/PubSubClient.h:93:4: note: PubSubClient::PubSubClient(uint8_t*, uint16_t, Client&, Stream&)
    PubSubClient(uint8_t *, uint16_t, Client& client, Stream&);
    ^
C:\Users\ladmin\Documents\Arduino\libraries\pubsubclient-master\src/PubSubClient.h:93:4: note:   candidate expects 4 arguments, 3 provided
C:\Users\ladmin\Documents\Arduino\libraries\pubsubclient-master\src/PubSubClient.h:92:4: note: PubSubClient::PubSubClient(uint8_t*, uint16_t, Client&)
    PubSubClient(uint8_t *, uint16_t, Client& client);
    ^
C:\Users\ladmin\Documents\Arduino\libraries\pubsubclient-master\src/PubSubClient.h:92:4: note:   no known conversion for argument 3 from 'void(char*, byte*, int) {aka void(char*, unsigned char*, int)}' to 'Client&'
C:\Users\ladmin\Documents\Arduino\libraries\pubsubclient-master\src/PubSubClient.h:91:4: note: PubSubClient::PubSubClient(IPAddress, uint16_t, void (*)(char*, uint8_t*, unsigned int), Client&, Stream&)
    PubSubClient(IPAddress, uint16_t, MQTT_CALLBACK_SIGNATURE,Client& client, Stream&);
    ^
C:\Users\ladmin\Documents\Arduino\libraries\pubsubclient-master\src/PubSubClient.h:91:4: note:   candidate expects 5 arguments, 3 provided
C:\Users\ladmin\Documents\Arduino\libraries\pubsubclient-master\src/PubSubClient.h:90:4: note: PubSubClient::PubSubClient(IPAddress, uint16_t, void (*)(char*, uint8_t*, unsigned int), Client&)
    PubSubClient(IPAddress, uint16_t, MQTT_CALLBACK_SIGNATURE,Client& client);
    ^
C:\Users\ladmin\Documents\Arduino\libraries\pubsubclient-master\src/PubSubClient.h:90:4: note:   candidate expects 4 arguments, 3 provided
C:\Users\ladmin\Documents\Arduino\libraries\pubsubclient-master\src/PubSubClient.h:89:4: note: PubSubClient::PubSubClient(IPAddress, uint16_t, Client&, Stream&)
    PubSubClient(IPAddress, uint16_t, Client& client, Stream&);
    ^
C:\Users\ladmin\Documents\Arduino\libraries\pubsubclient-master\src/PubSubClient.h:89:4: note:   candidate expects 4 arguments, 3 provided
C:\Users\ladmin\Documents\Arduino\libraries\pubsubclient-master\src/PubSubClient.h:88:4: note: PubSubClient::PubSubClient(IPAddress, uint16_t, Client&)
    PubSubClient(IPAddress, uint16_t, Client& client);
    ^
C:\Users\ladmin\Documents\Arduino\libraries\pubsubclient-master\src/PubSubClient.h:88:4: note:   no known conversion for argument 3 from 'void(char*, byte*, int) {aka void(char*, unsigned char*, int)}' to 'Client&'
C:\Users\ladmin\Documents\Arduino\libraries\pubsubclient-master\src/PubSubClient.h:87:4: note: PubSubClient::PubSubClient(Client&)
    PubSubClient(Client& client);
    ^
C:\Users\ladmin\Documents\Arduino\libraries\pubsubclient-master\src/PubSubClient.h:87:4: note:   candidate expects 1 argument, 3 provided
C:\Users\ladmin\Documents\Arduino\libraries\pubsubclient-master\src/PubSubClient.h:86:4: note: PubSubClient::PubSubClient()
    PubSubClient();
    ^
C:\Users\ladmin\Documents\Arduino\libraries\pubsubclient-master\src/PubSubClient.h:86:4: note:   candidate expects 0 arguments, 3 provided
C:\Users\ladmin\Documents\Arduino\libraries\pubsubclient-master\src/PubSubClient.h:67:7: note: PubSubClient::PubSubClient(const PubSubClient&)
 class PubSubClient {
       ^
C:\Users\ladmin\Documents\Arduino\libraries\pubsubclient-master\src/PubSubClient.h:67:7: note:   candidate expects 1 argument, 3 provided
no matching function for call to 'PubSubClient::PubSubClient(byte [4], int, void (&)(char*, byte*, int))'


All good.


Indeed I did! Just fixed the typo, thanks @benlovell.


nodeshots con s finale?


Please write down meaningful GIT commit messages!


Feel free to submit a pull request for these kind of changes :-)


um, typo?


typo defendats


This explanation really doesn't belong here.


That would make sense :-)


RAWK


Don't remove `phonebook_orig`, which is used for comparisons.


Also, let me know if there's anything I can do to help. I'd really like to see this issue fixed.


This is bugging me. With the floating point firmware, `t / 10000` returns of course a floating point value so the output is garbled, like 25.0625.0625

I assume it would look OK with integer firmware, but I don't really know.

And `256 % 10000` is 256, not 0256 as it should look. And this code really needs test cases, to prove that it works the way we expect it to. So I wrote one:

https://gist.github.com/nickandrew/c03a0e75bf8f1fd41745

All the test cases pass on the floating point firmware. I haven't tried the integer firmware (waiting for another ESP8266 in the mail).

```
OK   ... input 2000 output 125.0000
OK   ... input 1360 output 85.0000
OK   ... input 401 output 25.0625
OK   ... input 162 output 10.1250
OK   ... input 8 output 0.5000
OK   ... input 0 output 0.0000
OK   ... input 65528 output -0.5000
OK   ... input 65374 output -10.1250
OK   ... input 65135 output -25.0625
OK   ... input 64656 output -55.0000
```


This commit is being reviewed as a part of review issue #63.


You should ensure that both implementations (orig vs. opt) can be built with single `main.c`.


I think it's useful because that way you immedately know why those fields are created automatically and not e.g. by the user when managing the form. But I don't have strong feelings for that docstring so I wouldn't mind it going away...


Note that you can just set the following in hiera and it will globally disable the conntrack check :
```yaml
nagios::check::conntrack::ensure: 'absent'
```


:heart:


Why?


Yes, thanks :)


from the package


I know that getGenericInformation is more gramatically correct but does the API change implied is really worth it ? It implies a source-level break on client-code


The new version on master uses ov_fopen(). No restrictions for this function on Windows are mentioned. Why did'nt you consider to use this one as an alternative?


For using the nodeshot API with HTTPS :)


Search::init is only called when the program starts, before any UCI commands (such as the one to set Threads to 2) are processed. Also, isn't it already known that Min Split Depth = 2 is an ELO gainer at STC?


you will need a javascript tag
i.e.
@javascript


I believe this update broke compilation of PCM solver in the public version... I obtain the following error on a clean psi4public:

fatal: reference is not a tree: 9bc8e049f14260a4c1b4b2bda898a16850907b1c
Unable to checkout '9bc8e049f14260a4c1b4b2bda898a16850907b1c' in submodule path 'interfaces/pcmsolver'


You can put your user id and secret there: https://github.com/nhoizey/PinboardInGoogle/blob/master/PinboardInGoogle.user.js#L69-L70


@pajdak3 mjög gott, viljum halda projectinu okkar clean


Noticed.


fgdhdfgh


Do you think it is slow because we are doing a `__contains` query on an unindexed field? If we are getting mobile to change what they send to us (e.g. standardizing the type, sending app id, etc.), it might be easier if we run a query on an indexed column...
We could also think about purging old entries.


You are right, sorry


Whoops! Good spot, I will make these consistent


Can you provide a consistent function as the abstraction of the initialization of data allocation? Thus, we can avoid the further changes.


I think this may also be the original problem seen in this issue: https://github.com/psi4/psi4public/issues/183


Don't use `data` as naming scheme, which is too common to distinguish.


Sorry, I didn't notice but this is a wrong location. it overwrites 2.18. The proper place is in Dockerfile.txt template.

My fault.


Every time I see `camelize` my brain reads :candy:


Don't use `OPTIMIZE`, which might be reserved by toolchain.


Useless assignment to variable - `hash2`.


Method has too many lines. [14/10]


Align the parameters of a method call if they span more than one line.


lol


On voulait pas mettre les logs dans _debug aussi, ça pose un problème comme on fait un object.assign à ce moment non?
Pourquoi on renvoie le token au moment du send?


Don't leave the dead code.


Actually, let me review this some more. Not quite sorted out when I rebased on next...


The meaning of "optimal" and "optimizing" is a bit different.


Oh boy really thx!


no


Fix #113


I couldn't set the input field using fill_in


just checking if it makes any difference for ISE/quartus. It doesn't.

2015-10-06 17:00 GMT+02:00 Sébastien Bourdeauducq <notifications@github.com>
:

> What do you need the "full_case parallel_case" for?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/twlostow/urv-core/commit/014209e9c2e33c5fbdf5f40529237be36667fcf1#commitcomment-13616991>
> .
>


You are correct. Good catch!


this


that


Dit is een update functie, het enig wat gebruikt wordt van de sendable is de gebruiker's naam. Hier wordt dus data opgevraagd niet verstuurt!
De HunterInfoSendable.get() functie is eigenlijk een simpelere manier om de gegevens van de gebruiker hunter te krijgen.


k


Ok pour l'implémentation du nouveau header.


This commit is being reviewed as a part of review issue #63.


Weird results from leotrubach-7cores:

    4	leotrubach-7cores	seconds ago	59 / 1000	45	2	12	0	0	10.179


fixes #38


Make these choices into string constants?


I couldn't get it to work any other way, even merging authn-ldap/vendor with the globa /opt/conjur/vendor


Ternaries are not preferred for Python. You can set defaults in the calls to `add_argument` so that `args.port` always has a value be it the one provided or the one that is default.


Ok, agreed. Even ov_fopen() does not define which encodings are actually supported. Using the callback interface is the the only safe bet.


It's slow even without the `__contains`, actually - and I think as long as it comes last, that part might be okay.  We currently purge entries after 60 days.  I think we might need to store less information - I'd be interested in seeing what proportion of the logs are actually useful to people, maybe there are some we shouldn't even bother recording.  Otherwise, maybe it'd be best to keep only aggregate records - one entry per `log type, device, domain` with a counter or list of timestamps where it occurred.


Het is inderdaad fout om de laatste locatie te gebruiken, ipv de nieuwste. Ik was ervan uit gegaan dat getLastLocation() de laatste bekende locatie zou geven, dat is dus niet het geval. De HunterInfoSendable.get() zal ik aanpassen naar HunterInfoSendable.get(Location loc) zodat je de laatste locatie kan meegeven. De gebruiker's naam en mogelijk andere settings van de hunter kunnen met de preferences ingeladen worden in the get functie.


ok :+1:


A lot of this comes naturally when using good, declarative names. In your most recent version, you have named your predicates a lot more sensibly already: Declarative names (typically nouns and adjectives), indicating that these relations make sense in all directions. Nice work! Good luck with your genetic algorithm, I am looking forward to see how it performs in comparison with CLP(FD) constraints!


sounds good. that's what I'll do then. thx


I'd add the comment then where you generate the regform. And if we expect to have several RH in which regforms are going to be generated, I'd simply add a helper that generates them and add the comment there.


I love this thing.

also processing :+1:


```
[designate] pip install -r rackspace-requirements.txt
Obtaining file:./designate-dupdomain-hook#egg=dupdomainhook (from -r rackspace-requirements.txt (line 1))
    Complete output from command python setup.py egg_info:
    Traceback (most recent call last):
      File "<string>", line 18, in <module>
    IOError: [Errno 2] No such file or directory: './designate-dupdomain-hook/setup.py'

    ----------------------------------------
Command "python setup.py egg_info" failed with error code 1 in ./designate-dupdomain-hook
[designate]                              15:36:31  ☁  add_rack_requirements ☂ ✭
```

wat?  Am I missing something obvs @ionrock?


How about separating the file into two; run.sh and run0.bash.

In run.sh check the existence of bash.
If run.sh can find a installed bash, let's run.sh run bash with run0.bash.
run0.bash does the original testing.


fine for me


It is bytes/second. Token bucket.


Purging is disabled for smp tests so we cannot draw any conclusion from this test with that results.


Yea the idea is to just check against known valued and use that in the path instead of the $_GET being used directly in the path. I.e $resourceLoader->getLanguageKey() and $resourceLoader->getSkinName() or whatever the actual methods are.


Second `(flonum? x)` on this line should be `(flonum? y)` instead.


There are some cool options for that, there's the typical > thing the prof was using but there's also the option to use Latex: https://wiki.haskell.org/Literate_programming


And I just tested with the removal off os.path done by MiLo,
Indeed You're both are wright , guess that python manual makers should adapt there manuals.


This here is the actual fix.


@asedunov
Please take a look at this commit. New rule for names in backticks says that the following characters are forbidden: `/.;<>[]:\`, so I have replaced `\` with `!`.


Hardcoding this binary inside docker could be a no-no for the docker guys. I can image they are looking for something more dynamic to express hooks. I'm thinking about a run flag, or a config somewhere.
This way we are dictacting that for each container we will execute this binary to run hook binary inside the preconfigured hooks directory. What if I just want to run "echo "hello world"" before starting a container? should I create a specific .sh and put it into hooks.d?  or should I specify "bash -c "echo "hello world"" at container run?

While I like a central place and binary to execute defined hooks I think docker direction would be to go through having each command hook defined at run (or create, not sure about this). But I may be totally wrong. I can talk to maintainers to understand which way they're taking for this.


Forcing your preferred workflow on others is lame.
I happen to make my issues focused enough that they are the size of pull requests. I create new issues if I need to split up the work. Converting issues to pull-requests when I'm done is simply better for me. Boo on you!


There is no other code that returns a pointer. If you return the actual vector (and let it copy) it should work fine. That's what's done in all of the other places I could find.


The issue is ov_fopen() uses fopen() which requires a char* as local 8 bit.
Since Windows uses locale codepages for char* and not utf8, we cannot open Cyrillic paths on a German Windows.
This means to fix the bug, we must not use getLocalFileName().toLocal8Bit() on Windows.

The ov_open_callbacks API, allows us to use the QFile, which manages all this for us.


Do you think we should stop this test?


Wat is dit? :)


Most likely the checksum here is invalid. It should be `dad9aa4fd5364576534bb3b765631b5be5f0e1a24303857dce6915d01a297b92`, but it's better to verify it in case I'm wrong by running on the downloaded file:
```bash
shasum -a 256 ~/Downloads/couchbase-server-enterprise_4.0.0-elcapitan_x86_64.zip
```


thats cool ;-)


同时还修复了PostQRcode插件发送邮件不成功的bug


lgtm


For example in RPM, This:
```
# -- HTTPS Repositories ---------
Alias /pulp/repos /var/www/pub/yum/https/repos

<Directory /var/www/pub/yum/https>
    WSGIAccessScript /srv/pulp/repo_auth.wsgi
    SSLRequireSSL
    SSLVerifyClient optional_no_ca
    SSLVerifyDepth 2
    SSLOptions +StdEnvVars +ExportCertData +FakeBasicAuth
    Options FollowSymLinks Indexes
</Directory>
```

Changed to:
```
<Location  /pulp/repos/>
    RewriteEngine On
    RewriteRule (.+/pulp/repos/)(.+) /pulp/api/v2/content/redirect//var/www/pub/yum/https/repos/$2
</Location>
```


This causes $ruleType set at line 1244 to be overwritten by 'network' at line 1253 because that final else happens for all cases that are not 'any' or 'single'.
Edit a rule with source "LANnet" and you get the fields like this:
![network-lan](https://cloud.githubusercontent.com/assets/1535615/10313802/2fc1e258-6c71-11e5-9dfd-02dc180702d2.png)


Sure, that makes sense, I'll fix both comments


Ik snap niet waarom je een deelgebied variable hebt toegevoegd aan de ScoutingGroepInfo. In de API worden scouting groepen gesorteerd op id of je haal ze allemaal op. De deelgebieden variable is wordt dus nooit gezet. En kan ook nooit gezet worden omdat je niet weet welk deelgebied, de scouting groep van is.


I noticed that too but I hadn't thought much about it until you mentioned it. I looked them up and it does sound like they developed some interesting fantasy worlds and stories about them when they were children. The Wikipedia entry on one of their fictional worlds, Gondal, says that the characters and settings are sometimes taken from other sources that they enjoyed. ( https://en.wikipedia.org/wiki/Gondal_(fictional_country) ) Though they do exhibit signs of sci fi where the Brontes would not have been exposed to sci fi, it doesn't really sound like they made up tools and signs entirely from scratch. So I am not sure where Haas is going with her point.

I'm taking an Old English class and we have talked about how a lot of the languages and ideas that Tolkien used for Lord of the Rings came from Old English. He also incorporates elements of mythology and Catholicism into his other writings like the Silmarillion. I agree with you that Tolkien and the Brontes did something similar in their creation of unique and complex worlds, but I disagree with Haas that those types of writings come from scratch. Instead both Tolkien and the Brontes seem to reinforce her point that every decision and idea is based on a complicated web of cultural influences.


val can be empty


t


Same problem: `val` can be empty.


Hi, I think this change break the iOS7 compatibility, isn't it?


Looks good. Thank you.


I originally intended to only change the futilitymovecount at multiple threads.
But then I saw that then it will take more time for the first thread to split(). So I also reduced split depth.
Probably I should set splitdepth to 2 even in master. Then it is a valid test. Stopping it now...


This commit is being reviewed as a part of review issue #70.


What is the difference between this and poi_internet_access_wlan?


Is it possible to reduce resources needed for maintenance by back porting what is done in upstream already?

This patch looks too big and too far from the upstream standards by different reasons. For now we have serial, USB, watchdog, eMMC (w/o SDIO part), and GPIO are working in 4.2+. Why not using them?

The main issue with enabling this board in upstream is absence of the PMU (power management unit) driver. Could anyone do this driver properly and submit upstream? (In this patch it looks far away from the upstreaming point unfortunately.)

Next point is to make SDIO working since it's used by WiFi IP.

Seems maintaining this patch is quite a burden for anyone who is doing this.


@ligi Please note that the auto-formatter of Android Studio will change indentation again.


Thanks for the update Haig!
Can you please push the code that you've been trying?


is'nt => isn't


Hier weer het geval. Een scouting groep komt als volgend binnen:
{
   		adres: <heuevenseweg 1C 1234 AA Rheden>,
                id:  	<id>,
  		lat:     	<latitude>,
   		long:  	<longitude>,
   		naam: <naam>

 }
Er is dus geen aanduiding in welk deelgebied de groep zich bevindt. Je kan dus ook geen variable aan ScoutingGroepInfo toevoegen en verwachten dat het gezet wordt. Er niks bekend over welk deelgebied een ScoutingGroepInfo zit.


Oops, sorry for the typo


OK


Oh no, I was just wondering. This is your creation @thekordy - I'm not sure the scope you had in mind. You could have a vision that I'm not even aware of :)


Hmm.. this looked neater in terminal!


login里的delay这个东西感觉很难理解，不符合一般人对login的理解和使用习惯，delay还是保留，但默认改为delay=>0了 不影响你这边代码正常，只是知会下


It's a fix for the sha256sum check command line (one whitespace was missing).


Buena truji buena


Fixed here: https://github.com/netguru/access/commit/2b90bd8f2406bc4bbe1d3faddc5b4729bde25027


I think you will have to do container.RootPropagation=mount.RPRIVATE to retain the old behavior. Otherwise libcontainer will make / "rslave" by default.


I've made some tests and this AppVeyor is running bash.
Is it possible that its bash version does not implement Process Substitution?


excellent :+1:


@rhvgoyal thanks, was just going to ask you that!


Having said that, @rhatdan does want default beahavior to be "rslave" instead of "rprivate". So it might not be a bad thing, as long as it does not break any of the existing use cases.


I was also really surprised that many of the developers/designers disregarded the advocates. As @burke552 stated, it seems like Andrew was something that would be available to everyone on campus, so it seemed a bit reckless to ignore the suggestions made by the advocates.


Use this to be safe
o["Threads"]               << Option(2, 1, MAX_THREADS, on_threads);

Currently your search::init is called before o["Threads"] is changed by UCI.


Assertions must have messages explaining why the condition should have been true and what is the problem (e.g. show both parameter lists)


let's maintain the old behavior here, then in your PR you can adjust the value as you wish discussing this with others as well


Here is the patch I had written to keep the old behavior intact.

https://github.com/rhvgoyal/docker/commit/ba429627ccd8bfd7014dd93d56119debcc97a728


on daO278a


Add space after `for`


I think I played with removing these back in the days and the behavior changed so I guess we were getting `NaN`, `null` or `undefined` back in some situations? If that's not the case anymore with the new code, then this should go. Shall I remove it?


WHAT? +1


I'm pretty sure this is wrong, too.


Friend tell me what changes you have done, so i can merge them with mine.


I'm not sure whether this is the only relevant implementation of CallableDescriptor that must contain this information.

It also looks like a better approach if this would be computed once, when the descriptor is created, but I can't be sure


@rhvgoyal thanks, I've updated the code accordingly


Inverting condition of this `if` would decrease indent and improve readability


Actually this commit *does* close #1


@Kreegoth can you provide a bit more details on what happened? Was a bot loaded, what was it doing, a corpse was it available and so on, please?


Build is still broken. Please see http://ci.cgeo.org/job/c-geo/4075/console
Build was first broken at http://ci.cgeo.org/job/c-geo/4051/


I'm not talking about non pli4 based images.
If we take a look in MediaPlayer sources, we can see that there Hotplug always is included in the try. Hotplug is another plugin, so I think it is right. Therefore, I think it is wrong that in one place Hotplug is without try.
But of course, it is not very important.


@Rocky640
I believe you are correct.
Please make that fix ajithcj.


Are the items in `Items` ever updated?


1) The unit test cases for checkDate(), checkTask() and generateNextDate() have been done.
checkDate() returns a boolean value.
checkTask() returns a boolean value.
generateNextDate() returns a String object

2) assert statements and logging statements have been inserted into Flex.java, as required for the project.
(Even though at this stage, they are truly redundant.)

3) I have made Flex Display such that:
1. Whenever it displays error messages due to an invalid user input command, the Command Line Prompt will do the same.

2. If the user input command is valid, and it specifically requests for Tasks to be displayed (This means "searching", and "display" or "showing"), the Command Line Prompt will also display messages to notify the user of the valid input, and/or the action done.

3. If the user input command is valid, but it does NOT specifically requet for Tasks to be displayed (This means "add", "delete", "change" and "undo"), the same messages will be displayed on both Command Line Prompt, and Flex Display, to notify the user of the valid input, and/or action done.

As far as possible, all display messages have been converted into "non-magic" Strings, for refactoring purposes, as required by the project's code quality standard(s).


Agreed, I'll do it.


More tests needed:

- many parameters
- override that gives names to parameters (no errors expected in this case)
- two methods that overload each other (no overrides => no problem)
- generics: at least one supertype has a type parameter
- generics: the overridden function has its own generic parameters declared
- generics the subclass/subinterface that owns the inherited function is generic


Shouldn't this be more descriptive ?


Nothing, nothing... Just some testing with redis.


Any tests relevant for this?


Default environment often is stored in read-only memory so changing it would require re-flushing boot loader and its env. With this patch you can simply inject your extra logic by scripting in /uEnv.txt


```[WorkItem]```?


We have 2 cases:
1)internet_access without any "main" tag
2)internet_access with "main" (hotel, restaurant, cafe etc) tag.
In first case internet_access should be "main" tag (poi_type). So "internet_access_wlan" is such a tag.
In second case we do not need to split POI -> 1)"main tag" 2)internet_access. internet_access is a service offered by "main" POI. So we convert "internet_access" to internal tag "internet_access_type_wlan" in that case. This tag is poi_additional and is a part of "main" POI.
On the other side internet_access_wlan is a main tag which represents separate internet_access point.


blank line missing


What is the difference between the two `check()` functions? The names must explain it!


This one should leave in the backticks because docblocks can be parsed into HTML.


Lindo!


Just re using this commit for comments but.. What's going to be done for the submodule, now it clearly makes life much harder than the benefit of it is.. Feature branching is overcomplicated to handle now, and I dont think the submodules are something to use with project like this which actually consists of the "submodule" and most changes require commits to the base repo and submodule.


typo


Are other features implemented through `DeclaratiuonChecker` relevant to multi-declaratuions? If yes, how do they work?


Shouldn't the default be slightly more than two minutes? AFAIK this is the worst case duration until metadata is refreshed, so I'd wait a little longer.


Add a configuration option to enable `addLoggedInUserAsPerson` in this service.


comfort: 3
completeness: 5

Keypress exercise was hard, needed help from TA.


I am cross compiling from debian for arm. Gcc version 4.6.4. Before this mod all works fine. Now i have an error on WebServer.h at line 241 saying that map is not defined in namespace std. If i include Knownfile.h or SearchManager.h all works again. The reason is missing map definition. I cannot attach the error now because i am not at home. I will do later.


This commit is being reviewed as a part of review issue #70.


comfort: 3
completeness: 5

Struggled to make the winner bit a function, but managed it in my own way!


Sorry about that. Now fixed correctly.


comfort: 5
completeness: 5

Seems like it will make more sense as we do examples.


Tests on other invalid characters than `\:/` are missing


I'm not allowed to update `Items`.  From the description: "do not alter the Item class or Items property as those belong to the goblin in the corner who will insta-rage and one-shot you as he doesn't believe in shared code ownership"


Thanks for putting this.


This is not a link, it is an url.

Also, this is the gravatar url for the current user, so to prevent confusion you can use:

```ruby
def current_hacker_gravatar
  image_tag('http://www.gravatar.com/avatar/' + md5(current_hacker.email), class: 'avatar')
end
# or
def gravatar_url hacker
  'http://www.gravatar.com/avatar/' + md5(hacker.email)
end
```


al gefikst
dat komt t dus wel


t werkt


het werkt


Have you already tested this on master? I would feel more confident if the callback decoder for OGG files passes our new seek tests.


docs


In netdissect-stdinc.h, we have:
```
#if !(defined(_MSC_VER) && defined(UNALIGNED))
/* MSVC may have its own macro defined with the same name and purpose. */
#undef UNALIGNED
#define UNALIGNED       __attribute__((packed))
#endif
```
Perhaps a connection?


`$contenxt` has a typo. `$securityContext` is a better name, `$context` is too generic.


What about
```
package `foo/bar`
```

?


hey did godmode do this, or did you have to manually change it?


What about names of type parameters?


Did you add this file or is it added as part of the build process or something? :confused:


OK


Also please check parameter names in function types: `(x: Int) -> Unit`


typo


ok


This didn't fix anything... We are failing argument passing because the "Run didn't produce any output"


Would be a lot easier to read if this were declared closer to the use site


1) The unit test cases for checkDate(), checkTask() and generateNextDate() have been done.
checkDate() returns a boolean value.
checkTask() returns a boolean value.
generateNextDate() returns a String object

2) assert statements and logging statements have been inserted into Flex.java, as required for the project.
(Even though at this stage, they are truly redundant.)

3) I have made Flex Display such that:
1. Whenever it displays error messages due to an invalid user input command, the Command Line Prompt will do the same.

2. If the user input command is valid, and it specifically requests for Tasks to be displayed (This means "searching", and "display" or "showing"), the Command Line Prompt will also display messages to notify the user of the valid input, and/or the action done.

3. If the user input command is valid, but it does NOT specifically requet for Tasks to be displayed (This means "add", "delete", "change" and "undo"), the same messages will be displayed on both Command Line Prompt, and Flex Display, to notify the user of the valid input, and/or action done.

As far as possible, all display messages have been converted into "non-magic" Strings, for refactoring purposes, as required by the project's code quality standard(s).


If you do not install anything into `/usr/bin/`, you do not need to add that directory, either...


Yo voy a montar un odoo si tengo algo de tiempo y lo explica mejor Javier. para trastear un poco, aunque tambien puedes poner un servidor web y tener 10 meses de hosting gratis. O un server del minecraft, minig de bitcoins, etc ...


Nit: This seems a bit long for a 'one liner', and there's an extra space after `+=`


Can't see a test for empty backticks. I understand that it's prohubited by the lexer, but a test would make sense nevertheless


> Is it possible that its bash version does not implement Process Substitution?

... sigh.

o.k. I will rewrite the test with temporary files.

BTW, I would like to use bash in building process.
As I wrote in #544, I'm thiking about introducing a translator.


I am hitting this on NixOS, apparently XDG_RUNTIME_DIR is not being set by the default profile (non-X11).

Can't the suggestion of @bjornfor be implemented?


Hey! Unfortunately we tagged jest as 0.5.9 instead of 0.6.0. Please stick with 0.5.8 if you are using iojs or upgrade to node 4, which has support for arrow functions :) Sorry for the inconvenience.


`File loader accepts an option to set the inner-runner executable`


Yes, this will be one single file per date (date_recorded will only occur once)


Same arguments as for OGG files apply here. Porting the callback code to master and verifying that it passes the new seek tests would be great. I just noticed that we don't have a test file for WavPack and the file extension ".wv" is not listed in SoundSourceProxyTest.


The next agenda is adding a google maps activity and integrating the Google Maps API to make it a real time activity. Also, combining both the Login and Sign up activities into a single tabbed activity layout. Populating the listviews with real time data from the server using java and json modules.


Sorry, it's a bit confusing, but `UNALIGNED` doesn't mean "the individual members of this structure should not be aligned on natural boundaries - don't add padding to make them aligned", it means "this *entire structure* is not guaranteed to be aligned on a natural boundary, so don't generate code that assumes it's aligned".

I.e., if you have a packet format that doesn't put multi-byte integral or floating-point values on natural boundaries within the packet, don't rely on `UNALIGNED` to keep items on the appropriate boundaries.

Perhaps it's time for tcpdump to define types `nd_uint{8,16,24,32,48,56,64}` and `nd_int{8,16,24,32,48,56,64}`, which are just arrays of 1 through 8 `unsigned char`s, have all structures used for packet layouts use them and have *explicit* pad members as necessary, and require everybody to use the `EXTRACT_` macros to fetch fields from packets.  (Everybody should be using the `EXTRACT_` macros *anyway*, so as not to write code that expects to be running on a machine with a given byte order - or alignment.)


@jcelerier I would like to highlight this commit as it add some complexity inside a very simple part of code.
but this could be a first step to have Message that query stuff on the fly (see the last 3 posts into this Asana thread : https://app.asana.com/0/26437404589663/50157657302571 )


I'm not sure...


Remove the not used return comments


Not sure yet...


Thank you michal-konopinski !


Good catch, Pierre ;)  I am indeed going to implement a similar hierarchical SfM strategy. The main difference will be that Divide and Conquer sees great speed improvements in the matching process with some nice vocabulary tree tricks. I will only implement the hierarchical SfM portion and keep the matching the same as it is now


LGTM


And now leotrubach has 60 crashes. Can someone drop this machine from the pool?


ok add version number to the meta data

add data id to the data entries

Otherwise looks good


I think I'm growing to like the use of `?`. However I feel like we need to standardize its use across other options where it (`?`) might make sense.


Please review the commit message, see http://stackoverflow.com/questions/2290016/git-commit-messages-50-72-formatting and http://tbaggery.com/2008/04/19/a-note-about-git-commit-messages.html for more info.


Thanks @linquize I'll get a PR with a fix up asap.


Sorry, this bit of the lemma save us from that case

<code>
...
let x = fs_step ctxt1 (s0,fs0) in
  case x of
  None => True ...
</code>


@duoduoliu0622 you can calling the /statistics/advertiser/date entry point :
```php
return $client->api('statistics_advertiser')->date([
    'date-from' => date('Y-m-d', strtotime('-1 week')),
    'date-to' => date('Y-m-d'),
    'campaignid' => $campaignId,
]);
```


@stephanlindauer Is `<string name="chat_heading">messages will be deleted after 30 minutes.</string>` a server side setting?


Yep, good work. I hope I can add the unused import check to inspektor to start mitigating the problem with unused imports we're having.


@flavour This cause the style to 'null'. The gis_layer_feature prepop set the style, then the cap_warning_priority overrides the same. Should we have deduplicate config for gis_style or am I missing something here?


I prefer redundent this. :P


fixes #1


Closes #71


I guess you're using "precision" here as number of decimal digits ?
Why not using the C-API exposed semantic of "grid size", defaulting it to 0 meaning "floating precision" ?


I think that's referring to refactoring `Item`, I'm asking about actually -aging- the items in `Items`, and updating the `.Quality` & `.SellIn`. Since the updates made on `updatedItemList` aren't saved wouldn't each call to `app.UpdateQuality()` produce the same result?


Hi Kevin,

On Oct 6, 2015, at 14:49 , Kevin Darbyshire-Bryant <kevin@darbyshire-bryant.me.uk> wrote:

> On 05/10/15 21:43, Sebastian Moeller wrote:
>> Allow all cake overhead keywords to be used in any
>> order
>>
>> The overhead keywords for cake, basically symbolic names for overheads
>> commonly encountered at edge links to the home have had a peculiar
>> under documented feature in that the sequence of these options was
>> deciding which of the optins was used. But this behaviour did not
>> occur for all keywords (mainly to those that can be considered full
>> overhead specifications like pppoa-llc). A few keywords like via-ethernet,
>> ether-fcs and ether-vlan could would still be valid to modify the overhead
>> value given by the full-overhead keyword. This behaviour while consistent
>> is really difficult to explain to new users, so this commit changes the
>> logc of all keaywords to be additive, so all evaluated in place.
>> This allows non-sensical specifications like "pppoa-llc pppoe-llc"
>> to be used, but if the user asked for it they probably wanted it...
>> Note that the keyword "raw" will set the overhead to 0, but any keyword
>> value occurring later in the command is still applied; the same logic applies
>> to the "overhead N" option to specify the overhead numerically.
>>
>> Signed-off-by: Sebastian Moeller <moeller0@gmx.de>
>
> Ack.  Will wait a couple of days before applying whilst the API/ABI
> changes go in (or not) and deal with resulting issues if any.

No hurry, personally I prefer the �overhead N� option, but in sqm-scripts I also want to expose all the keywords (sure I could handle the current ordering requirement in sqm-scripts, but I believe the current state to be actively �hostile� and hence not in accord with cake�s mission of simplicity; to be precise I believe the symbolic keywords to be ugly in line with simplicity, just not the unexpected ordering effects).
	So before I go and expose those, I want cake to be in openwrt upstream ;) first, so I can wait a bit longer.

Best Regards
	Sebastian


I forgot to mention that I also updated the caching directives in htaccess to use the latest syntax and to (ostensibly) perform better.


not mine - as mine was changing it this way in the first place - we
really need some css like thingy for code :-) In the meantime do we have
some xml-thingy to import into android-studio so we all have the same
formatting-rules & this does not happen anymore?

On 06.10.2015 17:47, Tobias Preuss wrote:
> @ligi Please note that the auto-formatter of Android Studio will change indentation again.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/criticalmaps/criticalmaps-android/commit/6bf9ee50001aa14772d19c7ffb133393ab1fb3db#commitcomment-13618203
>

--
best regards,
ligi
http://ligi.de


Please use it in `acquire_host_info()` as well. Just make sure to set `host = NULL` in case of success at the end of the function, so the cleanup functions doesn't do anything.


Ah yes good idea.


Ah well, fair enough :)


How difficult is it to get DarkWeb-Everywhere included by default into TBB?


Comfort: 2.5
Completeness: 5

Would like to do more in class examples of closures and constructors vs. prototypes


We should use consistent spelling here - since we use "gray" in `edx_grayscale_neutral_xx_light`, we should use the same spelling in here as well (also in the base `grey_separator`).


まぁリストも[]モナドなのでモナドからは逃れられんが


What is the advantage of the above?


ping from Paris @ggreg ! Do you recall why raising an exception inside the future at this point? This breaks optional activity tasks (with `raises_on_failure=False`) and I don't see where it could be useful. And anyway, it's a big change in futures behaviour, maybe some debug thing you forgot to remove?


Nice docstring !


trailing whitespace


Download: https://dl.dropboxusercontent.com/u/21757911/source/vbsp-modified.zip


删除


Algunha idea de porque non funciona eso?

 public void clickCorto(View view){
        Intent intent=new Intent(this, SegundaActivity.class);
        startActivity(intent);
    }

teño a segunda clase creada co xml asociado e tamén o teño posto no manifest, pero cando lle dou o botón non fai nada.


Might want to add explanation of need for filename here?


This is better done with arguments passed to fgettext, like  `fgettext("Minimal Viewing Range: $1 nodes", get_vrange_min)`


all the commented content can probably be scrapped?


I sorta liked it in one line with `&&` because it's basically same check.


@Rocky640, my mistake. Thanks for the correction.
@VoyagerOne, pls review and approve(note I widened a bit more this time)


I use the default formatting settings of Android Studio. I set up a [macro on save](http://stackoverflow.com/a/5581992/356895).


For me, this shows up as nil


Thanks for catching that. Updated.


The bindings shouldn't expose handles — a caller can easily crash the process by passing in a bad value. This API also requires four JNI calls to do a get (`rawGet`, `rawMeta`, `rawBody`, `rawFree`).

Instead, please implement it something like this:
```java
public byte[][] rawGet(String storeName, String key);
```
This returns an array of two byte arrays; the first is the meta, the second is the body. Then you don't need the other methods.


It returns None on cancel, a list on success. So it's not an option to just return a vector by value. The part that did the same was Player::getAvailableSubtitle(Audio)Streams, which was removed in #7750. We even discussed that part.


`meta` should be a `byte[]`. It can contain binary data.


@wlbragg
You can try a variation of the reflectionmap, putting a bit less reflection on some areas. I uploaded a reflectionMap2.png on gdrive.  Please, test it and use/discard based on your experience.

Cheers, bro.


For the last 6 monthes, i've been strugglying with the state